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

Update14112021 #304

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

Update14112021 #304

wants to merge 9 commits into from

Conversation

kaleofr
Copy link

@kaleofr kaleofr commented Nov 14, 2021

No description provided.

@pep8speaks
Copy link

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

Line 82:61: W292 no newline at end of file

Line 17:80: E501 line too long (114 > 79 characters)
Line 20:80: E501 line too long (287 > 79 characters)

Line 8:80: E501 line too long (271 > 79 characters)

Line 21:9: E265 block comment should start with '# '
Line 21:80: E501 line too long (170 > 79 characters)
Line 22:42: E203 whitespace before ':'
Line 22:43: E701 multiple statements on one line (colon)
Line 22:80: E501 line too long (134 > 79 characters)
Line 22:81: E231 missing whitespace after ','
Line 22:94: E231 missing whitespace after ','
Line 22:105: E231 missing whitespace after ','
Line 22:120: E231 missing whitespace after ','
Line 25:13: E201 whitespace after '('
Line 25:44: E225 missing whitespace around operator
Line 25:80: E501 line too long (109 > 79 characters)
Line 25:83: E202 whitespace before ')'
Line 25:85: E701 multiple statements on one line (colon)
Line 25:93: E261 at least two spaces before inline comment
Line 27:31: E701 multiple statements on one line (colon)
Line 28:13: E701 multiple statements on one line (colon)
Line 29:35: E701 multiple statements on one line (colon)
Line 30:13: E701 multiple statements on one line (colon)
Line 31:33: E701 multiple statements on one line (colon)
Line 32:13: E701 multiple statements on one line (colon)
Line 34:43: E225 missing whitespace around operator
Line 34:57: E225 missing whitespace around operator
Line 34:80: E225 missing whitespace around operator
Line 34:80: E501 line too long (122 > 79 characters)
Line 34:89: E203 whitespace before ':'
Line 34:90: E701 multiple statements on one line (colon)
Line 34:98: E261 at least two spaces before inline comment
Line 35:43: E231 missing whitespace after ','
Line 35:50: E231 missing whitespace after ','
Line 35:54: E231 missing whitespace after ','
Line 35:59: E231 missing whitespace after ','
Line 37:34: E231 missing whitespace after ','
Line 37:41: E231 missing whitespace after ','
Line 37:45: E231 missing whitespace after ','
Line 37:50: E231 missing whitespace after ','
Line 38:9: E265 block comment should start with '# '
Line 38:80: E501 line too long (172 > 79 characters)
Line 39:73: E231 missing whitespace after ','
Line 40:55: E231 missing whitespace after ','
Line 41:57: E231 missing whitespace after ','
Line 42:57: E231 missing whitespace after ','
Line 43:38: E703 statement ends with a semicolon
Line 44:56: E231 missing whitespace after ','
Line 46:41: E231 missing whitespace after ','
Line 47:59: E231 missing whitespace after ','
Line 52:80: E501 line too long (81 > 79 characters)
Line 73:1: E305 expected 2 blank lines after class or function definition, found 1

Line 7:1: E302 expected 2 blank lines, found 1
Line 17:80: E501 line too long (80 > 79 characters)
Line 19:33: E221 multiple spaces before operator

Line 153:24: W292 no newline at end of file

@Commit-La-Grenouille
Copy link
Contributor

This PR has been rebased and adjusted to get a 100% successful CI: #326

It can be closed anytime between now and the time the other PR gets merged (or rejected).

1 similar comment
@Commit-La-Grenouille
Copy link
Contributor

This PR has been rebased and adjusted to get a 100% successful CI: #326

It can be closed anytime between now and the time the other PR gets merged (or rejected).

@f-laurens
Copy link
Contributor

To be closed, since superseded by #326 ?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants