]> review.fuel-infra Code Review - openstack-build/cinder-build.git/commitdiff
Make tox.ini run pep8/hacking checks on bin.
authorDan Prince <dprince@redhat.com>
Thu, 15 Nov 2012 16:27:06 +0000 (11:27 -0500)
committerDan Prince <dprince@redhat.com>
Thu, 15 Nov 2012 19:11:25 +0000 (14:11 -0500)
This updates the pep8 checks in our tox.ini file so that
we are also scanning the bin directory.

Additionally, it updates the main pep8 check so that it scans
cinder properly as well. Previously there were a bunch of files
getting skipped due to some of the pep8 pattern matching.

Change-Id: Ib47a590303409e62546a75625ee356c9fa7af6ad

tox.ini

diff --git a/tox.ini b/tox.ini
index a9edab8a923e2714cdbbfae0d83974f29ae54b54..afb9d3317e61660a6b0627a628e99909a6b3e99e 100644 (file)
--- a/tox.ini
+++ b/tox.ini
@@ -14,7 +14,11 @@ commands = /bin/bash run_tests.sh -N -P {posargs}
 
 [testenv:pep8]
 deps = pep8==1.1
-commands = pep8 --repeat --show-source cinder setup.py
+commands =
+  python tools/hacking.py --ignore=N4,E12,E711,E721 --repeat --show-source \
+      --exclude=.venv,.tox,dist,doc,openstack,*egg  .
+  python tools/hacking.py --ignore=N4,E12,E711,E721 --repeat --show-source \
+      --filename=cinder* bin
 
 [testenv:venv]
 commands = {posargs}