diff options
| author | John L. Villalovos <openstack.org@sodarock.com> | 2018-04-10 16:48:00 -0700 |
|---|---|---|
| committer | John L. Villalovos <openstack.org@sodarock.com> | 2018-04-10 16:48:26 -0700 |
| commit | ca91d4d871bae437da2c6173e7a0aad5c3b44532 (patch) | |
| tree | 75e2c268c054bf7df496d1c55891bb2cec359a3a /tox.ini | |
| parent | fc15be6e329a41ab4cf634c53a54441694c41195 (diff) | |
| download | ironic-ca91d4d871bae437da2c6173e7a0aad5c3b44532.tar.gz | |
Remove pycodestyle version pin. Add E402 and W503 to ignore.
Rather than pinning pycodestyle, ignore the E402 and W503 error. Fix
issue with E731 (Do not assign a lambda expression, use a def).
W503 is something we will likely never enable as it is a personal
style decision and can change depending on the code. There is no one
right answer. Interestingly there is also a W504 which is the opposite
check.
E402 is one we should most likely fix. But it can be done in a follow
patch or patches.
List of various error codes:
https://pycodestyle.readthedocs.io/en/latest/intro.html#error-codes
Change-Id: Ie2c31a8a3c75beeef22453cab5878cf0094bdf3f
Diffstat (limited to 'tox.ini')
| -rw-r--r-- | tox.ini | 5 |
1 files changed, 4 insertions, 1 deletions
@@ -101,7 +101,10 @@ setenv = PYTHONHASHSEED=0 commands = {posargs} [flake8] -ignore = E129 +# TODO(jlvillal): We should fix the E402 errors +# [E402] Module level import not at top of file +# [W503] Line break before binary operator. +ignore = E129,E402,W503 filename = *.py,app.wsgi exclude = .venv,.git,.tox,dist,doc,*lib/python*,*egg,build import-order-style = pep8 |