From: Christian Berendt <berendt@b1-systems.de>
Date: Thu, 31 Jul 2014 08:17:54 +0000 (+0200)
Subject: Add note that some checks are disabled on purpose
X-Git-Url: https://review.fuel-infra.org/gitweb?a=commitdiff_plain;h=3a04abfc5c9cd06ee229afd8bd4da4d932dd5244;p=openstack-build%2Fcinder-build.git

Add note that some checks are disabled on purpose

Change-Id: I9a2daf57a8bb73e292d99f5177c310d75061cc13
---

diff --git a/tox.ini b/tox.ini
index 07c7e66bf..87feb47c1 100644
--- a/tox.ini
+++ b/tox.ini
@@ -49,9 +49,29 @@ commands = {toxinidir}/tools/config/generate_sample.sh -b . -p cinder -o etc/cin
 commands = {posargs}
 
 [flake8]
-# Due to hacking 0.9.2 following checking are ignored on purpose for now
-#  E111,E112,E113,E121,E122,E123,E126,E128,E251,E265
-#  E713,F403,F841,H302,H305,H307,H402,H405,H803,H904
+# Following checks are ignored on purpose.
+#
+# E251 unexpected spaces around keyword / parameter equals
+# reason: no improvement in readability
+#
+# E265 block comment should start with '# '
+# reason: no improvement in readability
+#
+# H402 one line docstring needs punctuation
+# reason: removed in hacking (https://review.openstack.org/#/c/101497/)
+#
+# H803 git commit title should not end with period
+# reason: removed in hacking (https://review.openstack.org/#/c/101498/)
+#
+# H904 wrap long lines in parentheses instead of a backslash
+# reason: removed in hacking (https://review.openstack.org/#/c/101701/)
+#
+# Due to the upgrade to hacking 0.9.2 the following checking are
+# ignored on purpose for the moment and should be re-enabled.
+#
+# F841,H302,H305,H307,H405
+
+
 ignore = E251,E265,E711,E712,E713,F402,F841,H104,H302,H305,H307,H402,H405,H803,H904
 builtins = _
 exclude = .git,.venv,.tox,dist,tools,doc,common,*egg,build