Skip to content

Conversation

@aziz
Copy link
Contributor

@aziz aziz commented Feb 9, 2016

Thanks @noscript, see #7

@stevemao
Copy link
Member

stevemao commented Feb 9, 2016

What is your npm name? I'll add you so you can push yourself.

@aziz
Copy link
Contributor Author

aziz commented Feb 9, 2016

it's the same as my github username: aziz

aziz added a commit that referenced this pull request Feb 9, 2016
Using sed instead of tr  fixed #7
@aziz aziz merged commit 41b8b12 into so-fancy:master Feb 9, 2016
@aziz aziz deleted the patch-1 branch February 9, 2016 11:42
@aziz
Copy link
Contributor Author

aziz commented Feb 9, 2016

Bumped version and pushed to npm

@stevemao
Copy link
Member

stevemao commented Feb 9, 2016

Thanks @aziz :)

@rjc
Copy link

rjc commented Feb 9, 2016

I only just noticed that it's not a - character - should have read the linked issue.
Sorry for the noise.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants