From: John Griffith <john.griffith@solidfire.com>
Date: Tue, 14 May 2013 17:14:33 +0000 (-0600)
Subject: Fix up hacking ignores a bit.
X-Git-Url: https://review.fuel-infra.org/gitweb?a=commitdiff_plain;h=e136c969883e66917d95732e8765e50024848157;p=openstack-build%2Fcinder-build.git

Fix up hacking ignores a bit.

Add more explicit ignores to tox.ini hacking ignores until
we have a chance to fix up the imports and docstring errors
we've been ignoring.  This should give us everything we had
in the past (ie alphabetical import checks).

Also, update run_tests.sh to call flake8 and not the hacking.py
that is no longer there.

Change-Id: I6c82e958002255fd9f527b06bf0a68d148cc681e
---

diff --git a/run_tests.sh b/run_tests.sh
index 4f1155890..265d31dd3 100755
--- a/run_tests.sh
+++ b/run_tests.sh
@@ -110,12 +110,7 @@ srcfiles+=" setup.py"
 
 function run_pep8 {
   echo "Running PEP8 and HACKING compliance check..."
-  # Just run PEP8 in current environment
-  #
-
-  # Until all these issues get fixed, ignore.
-  ignore='--ignore=N4,E125,E126,E711,E712'
-  ${wrapper} python tools/hacking.py ${ignore} ${srcfiles}
+  flake8 cinder* bin
 }
 
 
diff --git a/tox.ini b/tox.ini
index ac75259b6..2e5e1707c 100644
--- a/tox.ini
+++ b/tox.ini
@@ -37,6 +37,6 @@ deps = -r{toxinidir}/tools/pip-requires
 commands = bash tools/lintstack.sh
 
 [flake8]
-ignore = E12,E711,E712,H3,H4,F
+ignore = E12,E711,E712,H302,H303,H304,H401,H402,H403,H404,F
 builtins = _
 exclude = .venv,.tox,dist,doc,openstack,*egg