Merge "Tox: add a pep8 target"

1 files changed, 9 insertions, 13 deletions

diff --git a/tox.ini b/tox.ini
index 6c568f3fe..882543d0c 100644
--- a/tox.ini
+++ b/tox.ini
@@ -1,7 +1,7 @@
 [tox]
 minversion = 2.0
 skipsdist = True
-envlist = py27,py3
+envlist = py27,py3,pep8
 
 [testenv]
 usedevelop=True
@@ -12,24 +12,20 @@ deps =
 commands = /bin/bash ./run_tests.sh
 whitelist_externals = /bin/bash
 
-
 [testenv:py27]
 # don't re-run coverage on both py27 py3, it takes too long
 setenv =
     SKIP_COVERAGE=1
 
-[flake8]
-# E125 is deliberately excluded. See https://github.com/jcrocholl/pep8/issues/126
-# The rest of the ignores are TODOs
-# New from hacking 0.9: E129, E131, H407, H405
-# E251 Skipped due to https://github.com/jcrocholl/pep8/issues/301
+[testenv:pep8]
+# for gate testing, scans only the files changed in the last commit
+commands =
+  /bin/bash tools/coding-checks.sh --pylint HEAD~ '{posargs}'
 
-# nova flake8 ignores
-#ignore = E121,E122,E123,E124,E125,E126,E127,E128,E129,E131,E251,H405
-# dovetail flake8 ignores
-ignore = E123,E125,H803
-max-line-length = 99
-exclude =  .venv,.git,.tox,dist,docs,*egg,build
+[testenv:pep8-full]
+# For manual testing, scans the whole codebase
+commands =
+  /bin/bash tools/coding-checks.sh --pylint '{posargs}'
 
 [testenv:os-requirements]
 commands =