Skip to content

Commit

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

Change-Id: Ieca721663aea2fd31753df4abfb5b01a7145b26a
  • Loading branch information
emonty committed May 16, 2013
1 parent 016a0b3 commit 967d929
Show file tree
Hide file tree
Showing 3 changed files with 14 additions and 5 deletions.
4 changes: 2 additions & 2 deletions openstackclient/shell.py
Expand Up @@ -24,7 +24,7 @@
from cliff import help

from openstackclient.common import clientmanager
from openstackclient.common.commandmanager import CommandManager
from openstackclient.common import commandmanager
from openstackclient.common import exceptions as exc
from openstackclient.common import openstackkeyring
from openstackclient.common import utils
Expand Down Expand Up @@ -63,7 +63,7 @@ def __init__(self):
super(OpenStackShell, self).__init__(
description=__doc__.strip(),
version=VERSION,
command_manager=CommandManager('openstack.cli'))
command_manager=commandmanager.CommandManager('openstack.cli'))

# This is instantiated in initialize_app() only when using
# password flow auth
Expand Down
7 changes: 6 additions & 1 deletion tools/test-requires
@@ -1,11 +1,16 @@
# 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

distribute>=0.6.24

coverage
discover
fixtures>=0.3.12
mock
openstack.nose_plugin
pep8==1.3.3
sphinx>=1.1.2
testrepository>=0.0.13
testtools>=0.9.26
8 changes: 6 additions & 2 deletions tox.ini
Expand Up @@ -11,8 +11,7 @@ deps = -r{toxinidir}/tools/pip-requires
commands = python setup.py testr --testr-args='{posargs}'

[testenv:pep8]
deps = pep8==1.3.3
commands = pep8 --ignore=E126,E202,W602 --repeat --show-source openstackclient setup.py
commands = flake8

[testenv:venv]
commands = {posargs}
Expand All @@ -22,3 +21,8 @@ commands = python setup.py testr --coverage --testr-args='{posargs}'

[tox:jenkins]
downloadcache = ~/cache/pip

[flake8]
ignore = E126,E202,W602,H402
show-source = True
exclude = .venv,.git,.tox,dist,doc,*openstack/common*,*lib/python*,*egg,build,tools

0 comments on commit 967d929

Please sign in to comment.