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

fork update #6

Open
wants to merge 6 commits into
base: master
Choose a base branch
from
Open

fork update #6

wants to merge 6 commits into from

Conversation

pynickle
Copy link
Owner

No description provided.

@pep8speaks
Copy link

pep8speaks commented Jun 14, 2019

Hello @pynickle! Thanks for updating this PR. We checked the lines you've touched for PEP 8 issues, and found:

Line 3:1: W191 indentation contains tabs
Line 3:2: E117 over-indented
Line 5:1: W191 indentation contains tabs
Line 5:2: E117 over-indented

Line 65:1: W191 indentation contains tabs
Line 65:2: E117 over-indented
Line 66:1: W191 indentation contains tabs
Line 66:3: E117 over-indented
Line 67:1: W191 indentation contains tabs
Line 68:1: W191 indentation contains tabs
Line 69:1: W191 indentation contains tabs
Line 69:3: E117 over-indented
Line 70:1: W191 indentation contains tabs
Line 88:1: E402 module level import not at top of file

Line 324:1: W191 indentation contains tabs
Line 324:1: W293 blank line contains whitespace
Line 325:1: W191 indentation contains tabs
Line 325:1: W293 blank line contains whitespace
Line 326:1: W191 indentation contains tabs
Line 326:1: W293 blank line contains whitespace
Line 327:1: E303 too many blank lines (3)
Line 327:1: E402 module level import not at top of file
Line 329:1: E302 expected 2 blank lines, found 0
Line 329:18: E231 missing whitespace after ':'
Line 329:26: E225 missing whitespace around operator
Line 330:1: W191 indentation contains tabs
Line 330:2: E117 over-indented
Line 331:1: E305 expected 2 blank lines after class or function definition, found 0
Line 331:22: E231 missing whitespace after ','

Comment last updated at 2019-06-14 12:44:48 UTC

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