Page MenuHomePhabricator

[python linting] apply aggressive mode in autopep8 (line wrapping)
ClosedPublic

Authored by markblundeberg on Feb 8 2020, 01:23.

Details

Summary

This adds the aggressive mode and then applies all sugggested
changes. Also I've removed the .autopep8 file, as I think it's
better to put things in the arclint file.

These are all just whitespace changes.

To fully reproduce these changes, install autopep8 1.5 and pycodestyle 2.5.0 (most
recent) which catches a few more things than the linux distro
versions. Older versions are happy with the result, but they
will suggest slightly fewer changes it seems.

Depends on D5215 D5216

Test Plan
ninja check-extended
arc lint --everything

Diff Detail

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

Event Timeline

markblundeberg created this revision.Feb 8 2020, 01:23
Herald added a reviewer: Restricted Project. · View Herald TranscriptFeb 8 2020, 01:23
markblundeberg edited the summary of this revision. (Show Details)Feb 8 2020, 01:27
markblundeberg edited the summary of this revision. (Show Details)Feb 8 2020, 01:41
deadalnix accepted this revision.Feb 8 2020, 01:54
This revision is now accepted and ready to land.Feb 8 2020, 01:54
markblundeberg edited the test plan for this revision. (Show Details)Feb 8 2020, 02:16