Page MenuHomePhabricator
Differential D388

Add autoPEP8 linter, use it on abc's tests.
ClosedPublic
Actions

Authored by deadalnix on Jul 28 2017, 14:08.

Details

Reviewers
freetrader
Group Reviewers
Restricted Project
Commits
rSTAGING6749c72e891a: Add autoPEP8 linter, use it on abc's tests.
rABC6749c72e891a: Add autoPEP8 linter, use it on abc's tests.
Summary

This doesn't do a perfect job, but it's still pretty damn good. Looks like it is worth adding to the toolchain.

Test Plan

Formatted ABC's RPC tests.

Diff Detail

Repository
rABC Bitcoin ABC
Branch
pep8
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 602
Build 602: arc lint + arc unit

Event Timeline

deadalnix created this revision.Jul 28 2017, 14:08
Herald added a reviewer: Restricted Project. · View Herald TranscriptJul 28 2017, 14:08
Harbormaster completed remote builds in B602: Diff 968.Jul 28 2017, 14:08
freetrader accepted this revision.Jul 28 2017, 14:52

All fine (tested)

This revision is now accepted and ready to land.Jul 28 2017, 14:52
Closed by commit rABC6749c72e891a: Add autoPEP8 linter, use it on abc's tests. (authored by deadalnix, committed by deadalnix). · Explain WhyJul 28 2017, 15:10
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
4 lines
arcanist/
2 lines
2 lines
linter/
79 lines
qa/
rpc-tests/
29 lines
73 lines
134 lines
15 lines

Diff 968

View Options

.arclint

Loading...
View Options

arcanist/.phutil_module_cache

Loading...
View Options

arcanist/__phutil_library_map__.php

Loading...
View Options

arcanist/linter/AutoPEP8Linter.php

Loading...
View Options

qa/rpc-tests/abc-cmdline.py

Loading...
View Options

qa/rpc-tests/abc-p2p-activation.py

Loading...
View Options

qa/rpc-tests/abc-p2p-fullblocktest.py

Loading...
View Options

qa/rpc-tests/abc-rpc.py

Loading...