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

Grammatical errors, spell corrections #460

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

davesque
Copy link

What was wrong?

Noticed errant uses of "it's" as well as some other low-hanging fruit around those errors.

How was it fixed?

Fixed incorrect uses of "it's", made some spelling corrections, and made some minor style improvements for text that was close by.

Cute Animal Picture

cute animal picture

@davesque
Copy link
Author

I actually noticed a lot more spelling errors and typos. It would be good to go back over the docs with some kind of automatic tools for that at some point.

@davesque
Copy link
Author

@carver @pipermerriam Any idea why this PR is failing on Travis? I can't gather much from the failure logs and I didn't make any changes to .py files outside of docstrings.

@pipermerriam
Copy link
Member

Because populus is broken on master. Likely the new rlp 1.0 release.

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.

2 participants