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

Test new updates #22

Closed
wants to merge 1 commit into from
Closed

Test new updates #22

wants to merge 1 commit into from

Conversation

OrkoHunter
Copy link
Collaborator

No description provided.

@OrkoHunter OrkoHunter closed this Jan 5, 2017
@OrkoHunter OrkoHunter deleted the OrkoHunter-patch-1 branch January 5, 2017 16:02
@OrkoHunter OrkoHunter restored the OrkoHunter-patch-1 branch January 5, 2017 16:02
@OrkoHunter OrkoHunter reopened this Jan 5, 2017
Copy link
Collaborator Author

@OrkoHunter OrkoHunter left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@pep8speaks fix all

@OrkoHunter OrkoHunter closed this Jan 5, 2017
@OrkoHunter OrkoHunter reopened this Jan 5, 2017
@OrkoHunter OrkoHunter closed this Jan 5, 2017
@OrkoHunter OrkoHunter reopened this Jan 5, 2017
@pep8speaks
Copy link
Collaborator

Hello @OrkoHunter, Thank you for updating man!

  • In the file test.py, following are the PEP8 issues :

Line 1:12: E401 multiple imports on one line
Line 1:17: E703 statement ends with a semicolon
Line 3:1: E302 expected 2 blank lines, found 1
Line 4:5: E265 block comment should start with '# '
Line 4:80: E501 line too long (83 > 79 characters)
Line 5:15: E225 missing whitespace around operator
Line 5:17: E201 whitespace after '('
Line 5:21: E231 missing whitespace after ','
Line 5:26: E231 missing whitespace after ','
Line 5:31: E202 whitespace before ')'
Line 5:33: E703 statement ends with a semicolon
Line 6:18: E225 missing whitespace around operator
Line 6:26: E231 missing whitespace after ':'
Line 6:80: E501 line too long (84 > 79 characters)
Line 7:5: E128 continuation line under-indented for visual indent
Line 7:12: E231 missing whitespace after ':'
Line 7:26: E231 missing whitespace after ','
Line 7:30: E231 missing whitespace after ','
Line 7:34: E231 missing whitespace after ','
Line 7:45: E231 missing whitespace after ','
Line 7:80: E501 line too long (83 > 79 characters)
Line 8:5: E122 continuation line missing indentation or outdented
Line 8:11: E231 missing whitespace after ':'
Line 8:20: E231 missing whitespace after ':'
Line 8:65: E231 missing whitespace after ','
Line 8:76: E231 missing whitespace after ':'
Line 9:5: E128 continuation line under-indented for visual indent
Line 9:7: E231 missing whitespace after ','
Line 9:11: E231 missing whitespace after ','
Line 9:17: E231 missing whitespace after ','
Line 9:27: E231 missing whitespace after ','
Line 11:1: E302 expected 2 blank lines, found 0
Line 11:50: E231 missing whitespace after ':'
Line 11:56: W601 .has_key() is deprecated, use 'in'
Line 11:69: E231 missing whitespace after ':'
Line 11:80: E501 line too long (85 > 79 characters)
Line 11:85: E703 statement ends with a semicolon
Line 12:1: E302 expected 2 blank lines, found 0
Line 12:16: E201 whitespace after '('
Line 12:25: E202 whitespace before ')'
Line 13:17: E211 whitespace before '('
Line 13:22: E201 whitespace after '('
Line 13:32: E202 whitespace before ')'
Line 14:6: E114 indentation is not a multiple of four (comment)
Line 14:6: E265 block comment should start with '# '
Line 15:6: E111 indentation is not a multiple of four
Line 15:13: E701 multiple statements on one line (colon)
Line 15:18: E225 missing whitespace around operator
Line 15:21: E702 multiple statements on one line (semicolon)
Line 15:27: E225 missing whitespace around operator
Line 15:31: E225 missing whitespace around operator
Line 15:39: E702 multiple statements on one line (semicolon)
Line 16:6: E111 indentation is not a multiple of four

Bye !

Copy link
Collaborator Author

@OrkoHunter OrkoHunter left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copy link
Collaborator Author

@OrkoHunter OrkoHunter left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@pep8speaks Suggest a diff

@pep8speaks
Copy link
Collaborator

Here you go with the gist !

You can ask me to create a PR against this branch with those fixes. Submit a review comment as @pep8speaks pep8ify.

@OrkoHunter

@OrkoHunter OrkoHunter closed this Jan 6, 2017
@OrkoHunter OrkoHunter deleted the OrkoHunter-patch-1 branch January 6, 2017 04:02
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