Add some more flake8 types to ignore list to fix the compilation errors
authorNuman Siddique <nusiddiq@redhat.com>
Wed, 6 Jan 2016 11:29:15 +0000 (16:59 +0530)
committerRussell Bryant <russell@ovn.org>
Wed, 6 Jan 2016 15:13:20 +0000 (10:13 -0500)
with the flake8 check enabled, ovs compilation is failing. This
patch adds few more flake8 types to the igore list.  These warnings
come from the hacking and docstrings flake8 plugins.

Signed-off-by: Numan Siddique <nusiddiq@redhat.com>
[russell@ovn.org added comments, also ignore W503]
Signed-off-by: Russell Bryant <russell@ovn.org>
Makefile.am

index 8b6ddb7..976522d 100644 (file)
@@ -342,14 +342,18 @@ endif
 
 if HAVE_FLAKE8
 ALL_LOCAL += flake8-check
+# http://flake8.readthedocs.org/en/latest/warnings.html
 # E123 closing bracket does not match indentation of opening bracket's line
 # E126 continuation line over-indented for hanging indent
 # E127 continuation line over-indented for visual indent
 # E128 continuation line under-indented for visual indent
 # E129 visually indented line with same indent as next logical line
 # E131 continuation line unaligned for hanging indent
+# W503 line break before binary operator
+# D*** -- warnings from flake8-docstrings plugin
+# H*** -- warnings from flake8 hacking plugin (custom style checks beyond PEP8)
 flake8-check: $(FLAKE8_PYFILES)
-       $(AM_V_GEN) if flake8 $^ --ignore=E123,E126,E127,E128,E129,E131 ${FLAKE8_FLAGS}; then touch $@; else exit 1; fi
+       $(AM_V_GEN) if flake8 $^ --ignore=E123,E126,E127,E128,E129,E131,W503,D100,D101,D102,D103,D104,D105,D200,D202,D203,D204,D205,D207,D208,D209,D210,D400,D401,H104,H201,H231,H232,H233,H238,H301,H306,H401,H403,H404,H405 ${FLAKE8_FLAGS}; then touch $@; else exit 1; fi
 endif
 
 include $(srcdir)/manpages.mk