Skip to content

1.11.1#38

Merged
heralden merged 241 commits intomasterfrom
dev
Dec 6, 2021
Merged

1.11.1#38
heralden merged 241 commits intomasterfrom
dev

Conversation

@julie-sullivan
Copy link
Copy Markdown
Member

Added organism to list upload.

@pep8speaks
Copy link
Copy Markdown

pep8speaks commented May 20, 2019

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

Line 95:13: E741 ambiguous variable name 'l'
Line 99:13: E741 ambiguous variable name 'l'

Line 866:1: W293 blank line contains whitespace
Line 926:21: E129 visually indented line with same indent as next logical line

Line 364:80: E501 line too long (82 > 79 characters)
Line 512:80: E501 line too long (80 > 79 characters)
Line 890:80: E501 line too long (85 > 79 characters)
Line 896:80: E501 line too long (81 > 79 characters)
Line 907:80: E501 line too long (81 > 79 characters)
Line 1456:55: W291 trailing whitespace

Line 107:80: E501 line too long (102 > 79 characters)
Line 108:80: E501 line too long (130 > 79 characters)
Line 166:80: E501 line too long (89 > 79 characters)
Line 176:80: E501 line too long (89 > 79 characters)
Line 333:80: E501 line too long (80 > 79 characters)
Line 349:80: E501 line too long (96 > 79 characters)
Line 407:80: E501 line too long (91 > 79 characters)
Line 408:80: E501 line too long (82 > 79 characters)
Line 409:80: E501 line too long (91 > 79 characters)
Line 410:80: E501 line too long (104 > 79 characters)
Line 655:80: E501 line too long (83 > 79 characters)

Line 121:80: E501 line too long (91 > 79 characters)
Line 165:80: E501 line too long (91 > 79 characters)
Line 212:80: E501 line too long (80 > 79 characters)

Line 353:80: E501 line too long (93 > 79 characters)
Line 374:80: E501 line too long (83 > 79 characters)
Line 377:80: E501 line too long (84 > 79 characters)
Line 626:80: E501 line too long (98 > 79 characters)
Line 846:80: E501 line too long (988 > 79 characters)
Line 863:33: E711 comparison to None should be 'if cond is not None:'
Line 866:80: E501 line too long (83 > 79 characters)
Line 879:34: E711 comparison to None should be 'if cond is not None:'
Line 882:80: E501 line too long (84 > 79 characters)
Line 998:80: E501 line too long (303 > 79 characters)
Line 1007:80: E501 line too long (303 > 79 characters)
Line 1016:80: E501 line too long (303 > 79 characters)

Line 34:80: E501 line too long (81 > 79 characters)
Line 188:80: E501 line too long (135 > 79 characters)
Line 197:80: E501 line too long (109 > 79 characters)
Line 201:80: E501 line too long (110 > 79 characters)

Comment last updated at 2020-10-13 13:54:56 UTC

@heralden heralden merged commit 646a29c into master Dec 6, 2021
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.