[PATCH 1/3] tox: Remove ignored PEP8 errors and directories
Stephen Finucane
stephen.finucane at intel.com
Fri Dec 11 07:27:46 AEDT 2015
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 <stephen.finucane at intel.com>
---
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
More information about the Patchwork
mailing list