pep8speaks
commented
2 years ago Hello @laurlee! Thanks for updating this PR. We checked the lines you've touched for PEP 8 issues, and found:
- In the file
solvation_analysis/plotting.py:
Line 30:53: E231 missing whitespace after ',' Line 30:56: E231 missing whitespace after ',' Line 37:1: E302 expected 2 blank lines, found 1 Line 41:1: E302 expected 2 blank lines, found 1 Line 59:1: E302 expected 2 blank lines, found 1 Line 76:120: E501 line too long (120 > 119 characters) Line 179:1: E302 expected 2 blank lines, found 1 Line 192:13: E128 continuation line under-indented for visual indent Line 194:1: E302 expected 2 blank lines, found 1 Line 235:1: E302 expected 2 blank lines, found 1 Line 256:1: E302 expected 2 blank lines, found 1 Line 291:1: E302 expected 2 blank lines, found 1 Line 304:1: W391 blank line at end of file
- In the file
solvation_analysis/tests/conftest.py:
Line 261:1: W391 blank line at end of file
- In the file
solvation_analysis/tests/test_plotting.py:
Line 20:1: E302 expected 2 blank lines, found 1 Line 25:1: E302 expected 2 blank lines, found 1 Line 31:1: E302 expected 2 blank lines, found 1 Line 32:40: E231 missing whitespace after ',' Line 37:1: E302 expected 2 blank lines, found 1 Line 38:40: E231 missing whitespace after ',' Line 43:1: E302 expected 2 blank lines, found 1 Line 49:1: E302 expected 2 blank lines, found 1 Line 54:1: E302 expected 2 blank lines, found 1 Line 59:1: E302 expected 2 blank lines, found 1 Line 63:1: W391 blank line at end of file
Description
Provide a brief description of the PR's purpose here.
Todos
Notable points that this PR has either accomplished or will accomplish.
Questions
Status