diff options
| author | Monty Taylor <mordred@inaugust.com> | 2013-05-18 09:40:15 -0700 |
|---|---|---|
| committer | Monty Taylor <mordred@inaugust.com> | 2013-05-18 09:40:15 -0700 |
| commit | b324ccc37bc6c0dfded22ba5b4eccc96642791ba (patch) | |
| tree | 246cad4d497bef7819e9701867d5156f23239f67 | |
| parent | f175a98e9fda8a68b592093040c2f7b8bd584d9b (diff) | |
| download | python-ceilometerclient-b324ccc37bc6c0dfded22ba5b4eccc96642791ba.tar.gz | |
Migrate to flake8.
Fixes bug 1172444.
Change-Id: Iba46d40a0e3ae0385ce51181f47295a2b520110d
| -rwxr-xr-x | run_tests.sh | 9 | ||||
| -rw-r--r-- | tools/test-requires | 8 | ||||
| -rw-r--r-- | tox.ini | 10 |
3 files changed, 14 insertions, 13 deletions
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 @@ -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 |
