From: Stephen Finucane <stephen@that.guru>
Date: Fri, 26 Oct 2018 09:39:29 +0000 (+0100)
Subject: tox: Disable W504 ("line break after binary operator")
X-Git-Tag: v2.2.0-rc1~233
X-Git-Url: http://git.ipfire.org/gitweb.cgi?a=commitdiff_plain;h=498a8eeaa6c9d1dca8ab29a845325cff1463e815;p=thirdparty%2Fpatchwork.git

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 384d3c7c..66c19214 100644
--- 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]