openstack/placement
Code Issues Proposed changes

Merge "update wording around pep8 exceptions"

Browse Source
This commit is contained in:
Jenkins
2016-08-04 19:26:00 +00:00
committed by Gerrit Code Review
parent 7ec2476b08 5a74149ec4
commit 07c449afb2
1 changed files with 12 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

15
tox.ini
View File

@@ -106,9 +106,18 @@ install_command = pip install -U --force-reinstall {opts} {packages}
commands = bash -c tools/releasenotes_tox.sh commands = bash -c tools/releasenotes_tox.sh
[flake8] [flake8]
# E125 is deliberately excluded. See https://github.com/jcrocholl/pep8/issues/126 # E125 is deliberately excluded. See
# The rest of the ignores are TODOs # https://github.com/jcrocholl/pep8/issues/126. It's just wrong.
# New from hacking 0.9: E129, E131, H407, H405 #
# Most of the whitespace related rules (E12* and E131) are excluded
# because while they are often useful guidelines, strict adherence to
# them ends up causing some really odd code formatting and forced
# extra line breaks. Updating code to enforce these will be a hard sell.
#
# H405 is another one that is good as a guideline, but sometimes
# multiline doc strings just don't have a natural summary
# line. Rejecting code for this reason is wrong.
#
# E251 Skipped due to https://github.com/jcrocholl/pep8/issues/301 # E251 Skipped due to https://github.com/jcrocholl/pep8/issues/301
ignore = E121,E122,E123,E124,E125,E126,E127,E128,E129,E131,E251,H405 ignore = E121,E122,E123,E124,E125,E126,E127,E128,E129,E131,E251,H405