jprdonnelly/impala
1
0
Fork 0
mirror of https://github.com/apache/impala.git synced 2025-12-19 18:12:08 -05:00
Code Issues Projects Releases Wiki Activity

Ignore flake8 W503 about breaking before operators

Browse Source 
PEP8 now reads that breaking before or after binary operators is fine,
and gives the rationale:

    But this can hurt readability in two ways: the operators tend to
    get scattered across different columns on the screen, and each
    operator is moved away from its operand and onto the previous
    line.

Change-Id: Ieedde356f613ce57caf3fe190fb44e0404337f97
Reviewed-on: http://gerrit.cloudera.org:8080/11196
Reviewed-by: Jim Apple <jbapple-impala@apache.org>
Tested-by: Impala Public Jenkins <impala-public-jenkins@cloudera.com>
This commit is contained in:
Jim Apple
2018-08-12 10:48:11 -07:00
committed by Impala Public Jenkins
parent 7002ad3b35
commit 0f6d051d85
1 changed files with 3 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
setup.cfg
View File

@@ -18,8 +18,10 @@
# E111,E114: These are ignored in order to support 2-space indents.
# E121,E125,E127,E128: ignored to allow some of the more common idioms for multi-line
# statement indentation used in the Impala code.
# W503: ignored to allow line breaks before binary operators, as is becoming standard:
# https://github.com/python/peps/commit/7690a00d478866
# E701: ignored to allow if statement body on same line as conditional
# https://pycodestyle.readthedocs.io/en/latest/intro.html#error-codes
ignore = E111,E114,E121,E125,E127,E128,E701
ignore = E111,E114,E121,E125,E127,E128,W503,E701
# Change from default to conform to Apache Impala line length.
max-line-length = 90