Closed robfairh closed 5 years ago
Hello @robfairh! Thanks for updating this PR. We checked the lines you've touched for PEP 8 issues, and found:
d3ploy/demand_driven_deployment_inst.py
:Line 186:56: W291 trailing whitespace Line 302:80: E501 line too long (90 > 79 characters) Line 309:80: E501 line too long (97 > 79 characters)
d3ploy/solver.py
:Line 202:1: E302 expected 2 blank lines, found 1 Line 206:58: W291 trailing whitespace Line 213:1: W293 blank line contains whitespace Line 221:46: W291 trailing whitespace Line 228:1: E302 expected 2 blank lines, found 1 Line 230:45: W291 trailing whitespace Line 243:1: W293 blank line contains whitespace Line 257:13: E722 do not use bare 'except' Line 258:77: W291 trailing whitespace Line 259:54: W291 trailing whitespace Line 262:29: E127 continuation line over-indented for visual indent Line 262:80: E501 line too long (84 > 79 characters)
d3ploy/supply_driven_deployment_inst.py
:Line 179:56: W291 trailing whitespace Line 294:80: E501 line too long (97 > 79 characters)
tests/integration_tests/decommission_test.py
:Line 21:78: E225 missing whitespace around operator Line 21:80: E501 line too long (95 > 79 characters) Line 45:1: E302 expected 2 blank lines, found 1
This is no longer necessary. This changes are addressed by #230
230 with change in config.yml