Skip to content
This repository has been archived by the owner on Jun 26, 2020. It is now read-only.

Commit

Permalink
Migrate to flake8.
Browse files Browse the repository at this point in the history
Fixes bug 1172444.

Change-Id: Iba46d40a0e3ae0385ce51181f47295a2b520110d
  • Loading branch information
emonty committed May 18, 2013
1 parent f175a98 commit b324ccc
Show file tree
Hide file tree
Showing 3 changed files with 14 additions and 13 deletions.
9 changes: 2 additions & 7 deletions run_tests.sh
Expand Up @@ -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"
Expand Down
8 changes: 6 additions & 2 deletions 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
Expand Down
10 changes: 6 additions & 4 deletions tox.ini
Expand Up @@ -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

0 comments on commit b324ccc

Please sign in to comment.