Skip to content

Commit

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

Change-Id: I4831946d01f05c2a10264745a07b53fa48be44c8
  • Loading branch information
emonty committed May 13, 2013
1 parent d9f7c21 commit a4fb55e
Show file tree
Hide file tree
Showing 3 changed files with 13 additions and 31 deletions.
24 changes: 0 additions & 24 deletions tools/flakes.py

This file was deleted.

8 changes: 6 additions & 2 deletions tools/test-requires
@@ -1,3 +1,9 @@
# 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

# Optional backend: SQL
pysqlite

Expand All @@ -14,8 +20,6 @@ nose # for test discovery and console feedback
nosexcover
openstack.nose_plugin
nosehtmloutput
pylint # static code analysis
pep8==1.3.3 # checks for PEP8 code style compliance
Sphinx>=1.1.2 # required to build documentation
unittest2 # backport of unittest lib in python 2.7
webtest # test wsgi apps without starting an http server
Expand Down
12 changes: 7 additions & 5 deletions tox.ini
Expand Up @@ -16,8 +16,8 @@ commands = nosetests {posargs}

[testenv:pep8]
commands =
pep8 --exclude=.venv,.tox,dist,doc,openstack,vendor,*egg --repeat --show-source .
pep8 --repeat --show-source --filename=keystone* bin
flake8
flake8 --filename=keystone* bin

[tox:jenkins]
downloadcache = ~/cache/pip
Expand All @@ -28,6 +28,8 @@ setenv = NOSE_WITH_COVERAGE=1
[testenv:venv]
commands = {posargs}

[testenv:pyflakes]
deps = pyflakes
commands = python tools/flakes.py keystone
[flake8]
show-source = true
ignore = H,F841,F401,F403
builtins = _
exclude=.venv,.git,.tox,dist,doc,*openstack/common*,*lib/python*,*egg,tools,vendor

0 comments on commit a4fb55e

Please sign in to comment.