Page MenuHomePhabricator

[LINTER] Enable new autopep8 rules

Authored by Fabien on Dec 18 2019, 10:40.



This enables rules that are disabled by default:

E226 - Fix missing whitespace around arithmetic operator.
E241 - Fix extraneous whitespace around keywords.
E242 - Remove extraneous whitespace around operator.

This is a follow-up from this request:

Test Plan

Should return no error:

arc lint --everything

Diff Detail

rABC Bitcoin ABC
Automatic diff as part of commit; lint not applicable.
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

markblundeberg added inline comments.
3 ↗(On Diff #14962)

probably better to have a newline here

3 ↗(On Diff #14962)

Good catch

This revision is now accepted and ready to land.Dec 18 2019, 17:15
This revision was automatically updated to reflect the committed changes.

(note that this also enabled W503, W504 which are disabled by default)