Differential D5219

[python linting] tweak options
ClosedPublic
Actions

Authored by markblundeberg on Feb 8 2020, 04:20.

Details

Reviewers

deadalnix

Group Reviewers

Restricted Project

Commits

rSTAGING336379264a2a: [python linting] tweak options
rABC336379264a2a: [python linting] tweak options

Summary

There's no need to specify --select when we're picking all the options.

It turns out that autopep8 with W50x options is a bit buggy, so disable that for now.
https://github.com/hhatto/autopep8/issues/518

Test Plan

arc lint --everything

(tried on flake8 3.5.0, 3.6.0, and 3.7.9 latest)

Diff Detail

Repository

rABC Bitcoin ABC

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

markblundeberg created this revision.Feb 8 2020, 04:20

Herald added a reviewer: Restricted Project. · View Herald TranscriptFeb 8 2020, 04:20

Build Bitcoin-ABC / Diffs / Diff Testing started.

markblundeberg added a parent revision: D5217: [python linting] apply aggressive mode in autopep8 (line wrapping).Feb 8 2020, 04:21

Build Bitcoin-ABC / Diffs / Diff Testing passed.

Harbormaster completed remote builds in B9351: Diff 16166.Feb 8 2020, 04:29

deadalnix accepted this revision.Feb 9 2020, 01:08

This revision is now accepted and ready to land.Feb 9 2020, 01:08

Closed by commit rABC336379264a2a: [python linting] tweak options (authored by Mark Lundeberg <markblundeberg@users.noreply.github.com>). · Explain WhyFeb 9 2020, 02:25

This revision was automatically updated to reflect the committed changes.

Mark Lundeberg <markblundeberg@users.noreply.github.com> added a commit: rABC336379264a2a: [python linting] tweak options.

Mark Lundeberg <markblundeberg@users.noreply.github.com> added a commit: rSTAGING336379264a2a: [python linting] tweak options.Apr 7 2020, 22:55

Revision Contents
Changeset List

Path

Size

.arclint

5 lines

Diff 16182

View Options

[python linting] tweak optionsClosedPublicActions