diff --git a/run_tests.sh b/run_tests.sh index aea2461..01ed56b 100755 --- a/run_tests.sh +++ b/run_tests.sh @@ -102,13 +102,8 @@ function copy_subunit_log { } function run_pep8 { - echo "Running pep8 ..." - srcfiles="--exclude=.venv,.git,.tox,dist,doc,*openstack/common*,*lib/python*" - srcfiles+=",*egg,build ." - # Just run PEP8 in current environment - # - ignore="--ignore=E12,E711,E721,E712" - ${wrapper} pep8 ${ignore} --show-source ${srcfiles} + echo "Running flake8 ..." + ${wrapper} flake8 } TESTRTESTS="testr run --parallel $testropts" diff --git a/tools/test-requires b/tools/test-requires index 06f5bd9..c4ca04d 100644 --- a/tools/test-requires +++ b/tools/test-requires @@ -1,10 +1,14 @@ -# This file is managed by openstack-depends +# Install bounded pep8/pyflakes first, then let flake8 install +pep8==1.4.5 +pyflakes==0.7.2 +flake8==2.0 +hacking>=0.5.3,<0.6 + coverage>=3.6 discover distribute>=0.6.24 fixtures>=0.3.12 mox>=0.5.3 -pep8==1.4.5 python-subunit sphinx>=1.1.2 testrepository>=0.0.13 diff --git a/tox.ini b/tox.ini index 27c8600..67b21ed 100644 --- a/tox.ini +++ b/tox.ini @@ -17,13 +17,15 @@ sitepackages = True downloadcache = ~/cache/pip [testenv:pep8] -deps = pep8==1.4.5 -commands = - pep8 --ignore=E12,E711,E721,E712 --show-source \ - --exclude=.venv,.git,.tox,dist,doc,*openstack/common*,*lib/python*,*egg,build . +commands = flake8 [testenv:cover] commands = python setup.py testr --coverage --testr-args='{posargs}' [testenv:venv] commands = {posargs} + +[flake8] +ignore = E12,E711,E721,E712,F,H +show-source = True +exclude = .venv,.git,.tox,dist,doc,*openstack/common*,*lib/python*,*egg,build