From e136c969883e66917d95732e8765e50024848157 Mon Sep 17 00:00:00 2001 From: John Griffith Date: Tue, 14 May 2013 11:14:33 -0600 Subject: [PATCH] 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 --- run_tests.sh | 7 +------ tox.ini | 2 +- 2 files changed, 2 insertions(+), 7 deletions(-) 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 -- 2.45.2