zuul
/
zuul
Code Issues Proposed changes

Ignore pep8 E123. 

From the pep8 spec:

  The closing brace/bracket/parenthesis on multi-line constructs
  may either line up under the last item of the list, as in:
  ...
  or it may be lined up under the first character of the line that
  starts the multi-line construct, as in:
  ...

Bizzarly, the pep8 tool ignores the first option.

Change-Id: Ic0e8f8ad964881276d213ffd50deb75a537c250b
Reviewed-on: https://review.openstack.org/28609
Reviewed-by: Clark Boylan <clark.boylan@gmail.com>
Approved: James E. Blair <corvus@inaugust.com>
Tested-by: Jenkins
This commit is contained in:
James E. Blair 2013-05-08 13:32:10 -07:00 committed by Jenkins
parent 51fd119902
commit 88dd5109c8
1 changed files with 1 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
tox.ini
View File

@ -14,7 +14,7 @@ downloadcache = ~/cache/pip
[testenv:pep8]
deps = pep8==1.3.3
commands = pep8 --ignore=E125,E128 --repeat --show-source --exclude=.venv,.tox,dist,doc,build .
commands = pep8 --ignore=E123,E125,E128 --repeat --show-source --exclude=.venv,.tox,dist,doc,build .
[testenv:cover]
setenv = NOSE_WITH_COVERAGE=1