Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Spellings #10

Merged
merged 2 commits into from
Nov 11, 2015
Merged

Spellings #10

merged 2 commits into from
Nov 11, 2015

Conversation

jameshartt
Copy link
Contributor

My apologies for a really nit picky PR that doesn't really add value, this it really just an opportunity for me to say thank you for creating a really helpful utility and I hope to jump on it properly in the near future. :-)

@marmelroy
Copy link
Owner

Thanks for noticing that James. Oops. I'll merge and update the test too so Travis is happy.

marmelroy added a commit that referenced this pull request Nov 11, 2015
@marmelroy marmelroy merged commit b5467c6 into marmelroy:master Nov 11, 2015
@jameshartt
Copy link
Contributor Author

Ahhh yes, I forgot about wee Travis. Good work :-)

@jameshartt
Copy link
Contributor Author

I should have had a look at the examples project, didn't notice the tests there :-)

@marmelroy
Copy link
Owner

No worries. I'll probably move most of the tests to the main project in the next release anyway....

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.

None yet

2 participants