tox: Disable W504 ("line break after binary operator")

This was introduced in a recent version of 'pycodestyle'. The
documentation notes [1] that it is mutually exclusive with W503, which
we do enforce, suggesting that we disable one or the other. Avoid the
churn and stick to the older rule, which I personally prefer.

[1] http://pycodestyle.pycqa.org/en/latest/intro.html#error-codes

Signed-off-by: Stephen Finucane <stephen@that.guru>

diff --git a/tox.ini b/tox.ini
index 384d3c7c3c54ec7667673394905817b9475ade3d..66c192146bbcfc5143ae4ab8b7884d3120d5f2b3 100644 (file)
--- a/tox.ini
+++ b/tox.ini
@@ -43,7 +43,12 @@ deps = flake8
 commands = flake8 {posargs} patchwork patchwork/bin/pwclient
 
 [flake8]
-ignore = E129, F405
+# Some rules are ignored as their use makes the code more difficult to read:
+#
+# E129 visually indented line with same indent as next logical line
+# F405 'name' may be undefined, or defined from star imports: 'module'
+# W504 line break after binary operator
+ignore = E129, F405, W504
 exclude = ./patchwork/migrations
 
 [testenv:docs]