Remove the following 'ignore' errors: E121-128,131 various visual indentation errors E251 spaces around keyword / parameter equals H405 multi-line docstring must have summary line
The only error remaining is E129, which is technically not part of the PEP8 standard and contributes in practice besides lots of manual line wrapping. In addition, there's no good reason why the tests directory shouldn't be assessed like everything else - tests are still code and, as such, should be held to the same standards.. Finally, include the 'pwclient' script which should not be ignored. Signed-off-by: Stephen Finucane <[email protected]> --- tox.ini | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/tox.ini b/tox.ini index a5bd898..77c450b 100644 --- a/tox.ini +++ b/tox.ini @@ -25,11 +25,11 @@ passenv = [testenv:pep8] basepython = python2.7 deps = flake -commands = flake8 {posargs} patchwork +commands = flake8 {posargs} patchwork patchwork/bin/pwclient [flake8] -ignore = E121,E122,E123,E124,E125,E126,E127,E128,E129,E131,E251,H405 -exclude = ./patchwork/tests +ignore = E129 +exclude = ./patchwork/migrations [testenv:lint] basepython = python2.7 -- 2.0.0 _______________________________________________ Patchwork mailing list [email protected] https://lists.ozlabs.org/listinfo/patchwork
