-
Notifications
You must be signed in to change notification settings - Fork 60
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
Capitalized, clean up #24
Conversation
@RichardLitt you broke my tests! |
...hooray? I'm guessing you want me to fix those, too? |
probably, yeah. Its just adjusting the expected message in the test thats failing |
Tests were having other problems too (see #34) |
a3ff664
to
69d664b
Compare
I removed all uppercase and final periods for |
@RichardLitt could you rebase this on master? |
69d664b
to
a1a49b2
Compare
@whyrusleeping Done. |
@RichardLitt youre gonna have to update the tests since you changed the output theyre checking against. |
a1a49b2
to
bb050f8
Compare
@whyrusleeping Passed! |
No description provided.