[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