Skip to content

Conversation

@brannondorsey
Copy link
Contributor

Addresses #39 + updates to README to reflect changes past and present.

@pep8speaks
Copy link

pep8speaks commented Mar 12, 2018

Hello @brannondorsey! Thanks for updating the PR.

Line 207:80: E501 line too long (90 > 79 characters)

Comment last updated on March 15, 2018 at 19:42 Hours UTC

@coveralls
Copy link

coveralls commented Mar 12, 2018

Coverage Status

Coverage remained the same at 94.053% when pulling 622949c on brangerbriz:master into 17d5119 on alecalve:master.

@brannondorsey
Copy link
Contributor Author

@alecalve, would you like me to clean up some of these PEP warnings before merging?

@alecalve
Copy link
Owner

That'd be great!

@brannondorsey
Copy link
Contributor Author

I've fixed all but one of the PEP formatting errors. Couldn't think of an elegant way to fix the last without changing code around too much so I figured ¯_(ツ)_/¯. Should be ready to merge.

@alecalve alecalve merged commit a2c7b50 into alecalve:master Mar 15, 2018
@alecalve
Copy link
Owner

Thanks again!

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.

4 participants