Page MenuHomePhabricator
Differential D3703

test: Move linters to test/lint, add readme
ClosedPublic
Actions

Authored by Fabien on Jul 22 2019, 09:24.

Details

Reviewers
deadalnix
jasonbcox
Group Reviewers
Restricted Project
Commits
rSTAGINGcebe40b92ba3: test: Move linters to test/lint, add readme
rABCcebe40b92ba3: test: Move linters to test/lint, add readme
Summary

Backport of core PR13281
https://github.com/bitcoin/bitcoin/pull/13281/files

Not ported:

  • Travis changes, irrelevant
  • Functional tests doc update, as we don't use a script for the python

linter but arcanist.

Test Plan

Read the doc.

Diff Detail

Repository
rABC Bitcoin ABC
Branch
PR13281
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 6898
Build 11843: Bitcoin ABC Buildbot (legacy)
Build 11842: arc lint + arc unit

Event Timeline

Fabien created this revision.Jul 22 2019, 09:24
Herald added a reviewer: Restricted Project. · View Herald TranscriptJul 22 2019, 09:24
Herald added a reviewer: deadalnix. · View Herald Transcript
teamcity added a comment.Jul 22 2019, 09:24

Build Bitcoin-ABC / Diffs / Bitcoin-ABC Diff Testing started.

teamcity added a comment.Jul 22 2019, 09:35

Build Bitcoin-ABC / Diffs / Bitcoin-ABC Diff Testing passed.

Harbormaster completed remote builds in B6898: Diff 10368.Jul 22 2019, 09:35
Fabien updated this revision to Diff 10369.Jul 22 2019, 09:43

Move check-rpc-mappings.py to test/lint.

teamcity added a comment.Jul 22 2019, 09:43

Build Bitcoin-ABC / Diffs / Bitcoin-ABC Diff Testing started.

teamcity added a comment.Jul 22 2019, 09:49

Build Bitcoin-ABC / Diffs / Bitcoin-ABC Diff Testing passed.

Harbormaster completed remote builds in B6899: Diff 10369.Jul 22 2019, 09:49
jasonbcox accepted this revision.Jul 22 2019, 20:06
This revision is now accepted and ready to land.Jul 22 2019, 20:06
Closed by commit rABCcebe40b92ba3: test: Move linters to test/lint, add readme (authored by Fabien). · Explain WhyJul 23 2019, 10:33
This revision was automatically updated to reflect the committed changes.
Fabien added a commit: rABCcebe40b92ba3: test: Move linters to test/lint, add readme.
Fabien added a commit: rSTAGINGcebe40b92ba3: test: Move linters to test/lint, add readme.Sep 19 2019, 01:22

Revision Contents
Changeset List

PathSize
contrib/
devtools/
23 lines
doc/
2 lines
test/
lint/
21 lines
test/
lint/
contrib/
devtools/
77 lines

Diff 10368

View Options

contrib/devtools/README.md

Loading...
View Options

contrib/devtools/git-subtree-check.sh

Loading...
View Options

doc/developer-notes.md

Loading...
View Options

test/lint/README.md

Loading...
View Options

test/lint/git-subtree-check.sh

Loading...