Check for pep8 E226 and E24

By default, pep8.py ignores E226 and E24. By changing
tox.ini to not have an ignore clause, it started using
the default. Since we were explicitely ignoring things
until recently, E226 and E24 where being checked. This
patch adds an ignore clause to tox.ini and fixes some
E226 errors to match our pep8 checks to other projects
in openstack.

Change-Id: Ia0bc0e6bcb4b4c6e54d61bae2fd7cad48303a62a
This commit is contained in:
David Peraza 2014-01-07 10:38:55 -06:00
parent 2245becc28
commit 7c9af12937
2 changed files with 3 additions and 2 deletions

View File

@ -375,8 +375,8 @@ def setup_events(trait_plugin_mgr):
errmsg = (_("Invalid YAML syntax in Event Definitions file "
"%(file)s at line: %(line)s, column: %(column)s.")
% dict(file=config_file,
line=mark.line+1,
column=mark.column+1))
line=mark.line + 1,
column=mark.column + 1))
else:
errmsg = (_("YAML error reading Event Definitions file "
"%(file)s")

View File

@ -43,6 +43,7 @@ deps = -r{toxinidir}/requirements.txt
commands = {posargs}
[flake8]
ignore = None
builtins = _
exclude=.venv,.git,.tox,dist,doc,./ceilometer/openstack/common,*lib/python*,*egg,tools,nova_tests,build
show-source = True