search

rdemaria / pysixtrack

Reference sixtrack tracking engine in pure python
Apache License 2.0
1 stars 12 forks source link

Fixes #20

Closed giadarol closed 5 years ago

pep8speaks commented 5 years ago

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

Line 19:38: W291 trailing whitespace Line 38:10: E128 continuation line under-indented for visual indent Line 53:45: E251 unexpected spaces around keyword / parameter equals Line 53:47: E251 unexpected spaces around keyword / parameter equals Line 54:45: E251 unexpected spaces around keyword / parameter equals Line 54:47: E251 unexpected spaces around keyword / parameter equals Line 58:1: E303 too many blank lines (3) Line 75:80: E501 line too long (94 > 79 characters) Line 154:1: W391 blank line at end of file

Line 19:38: W291 trailing whitespace Line 38:10: E128 continuation line under-indented for visual indent Line 53:45: E251 unexpected spaces around keyword / parameter equals Line 53:47: E251 unexpected spaces around keyword / parameter equals Line 54:45: E251 unexpected spaces around keyword / parameter equals Line 54:47: E251 unexpected spaces around keyword / parameter equals Line 58:1: E303 too many blank lines (3) Line 75:80: E501 line too long (94 > 79 characters) Line 122:28: E225 missing whitespace around operator Line 154:1: W391 blank line at end of file

Line 29:14: E231 missing whitespace after ',' Line 29:19: E225 missing whitespace around operator Line 29:61: W291 trailing whitespace

Line 29:14: E231 missing whitespace after ',' Line 29:19: E225 missing whitespace around operator Line 29:61: W291 trailing whitespace

Line 42:12: E225 missing whitespace around operator Line 43:25: E225 missing whitespace around operator Line 48:13: E225 missing whitespace around operator

Line 25:22: E231 missing whitespace after ',' Line 25:26: E231 missing whitespace after ',' Line 26:13: E225 missing whitespace around operator Line 28:19: E225 missing whitespace around operator Line 28:36: E231 missing whitespace after ',' Line 29:18: E225 missing whitespace around operator Line 31:27: E225 missing whitespace around operator

Comment last updated at 2019-06-18 16:10:08 UTC
giadarol commented 5 years ago

All examples are fixed except the sps one, which I don't fully understand. I leave it to @rdemaria :-)