openstack
/
os-brick
Code Issues Proposed changes

Ignore pep8 W503/W504 

W503 has been reversed in current pep8, so we
shouldn't start enforcing this here.

W504 takes the opposite stance of W503.

Neither of these seem worth enforcing in our
code for now, just punt on a strict rule for
this and aim for readable code.

Change-Id: Id38b2629ea22792bbc7b4dce77fa9e7039be140f
(cherry picked from commit 80e37a3e61)
This commit is contained in:
Eric Harney 2019-04-15 12:05:34 -04:00 committed by Brian Rosmaita
parent 15160ab610
commit 61346f1002
1 changed files with 9 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

10
tox.ini
View File

@ -84,8 +84,16 @@ commands = sphinx-build -a -E -W -d releasenotes/build/doctrees -b html releasen
#
# E251 unexpected spaces around keyword / parameter equals
# reason: no improvement in readability
# W503 line break before binary operator
# reason: pep8 itself is not sure about this one and
# reversed this rule in 2016
# W504 line break after binary operator
# reason: no agreement on this being universally
# preferable for our code. Disabled to keep checking
# tools from getting in our way with regards to this.
#
show-source = True
ignore = E251
ignore = E251,W503,W504
enable-extensions=H106,H203,H204,H205
builtins = _
exclude=.venv,.git,.tox,dist,*lib/python*,*egg,build