Skip to content
This repository has been archived by the owner on May 16, 2024. It is now read-only.

Disable lint W503 (break around binary operators) because doesn't conform PEP-8 #552

Closed
luisg123v opened this issue May 12, 2018 · 4 comments

Comments

@luisg123v
Copy link
Contributor

Lint W503 is triggered when a line break is before a binary operator, instead of being after, e.g.:

if (condition_1
        and condition_2):

However, PEP-8 has updated its position regarding this formatting, and it explicitly recommends quite the opposite, which is breaking before rather than after binary operators, like in the above example.

Therefore, the lint W503 should be disabled, because it goes against PEP-8 recomendations.

For more information, see:
PyCQA/pycodestyle#498

@luisg123v
Copy link
Contributor Author

CC @moylop260

@moylop260
Copy link
Contributor

It was fixed from #546

@luisg123v
Copy link
Contributor Author

@moylop260
As you can see in This Travis from OCA/pylint-odoo#196, the lint W503 is being triggered.

Doesn't it mean this is still valid? Sorry if I'm misunderstanding

@moylop260
Copy link
Contributor

OCA/pylint-odoo is not using MQT project.
You can ignore these checks from travis.yml:

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants