Page MenuHomePhabricator

Integrate the linter into arcanist

Authored by Fabien on Feb 9 2019, 08:51.


Group Reviewers
Restricted Project
rABCbe1efa0f4f36: Integrate the linter into arcanist
  • Make the script take a single file at input for name mismatch checking
  • Add a new arcanist linter that runs and parse the output
  • Make the script return 0 to avoid arcanist dropping the linter due to failure

Depends on D2532

Test Plan

Run arc lint, check there is no error.

Modify src/wallet/wallet_crypto_tests.cpp:
On the line containing BOOST_FIXTURE_TEST_SUITE, replace the test name:
wallet_crypto_tests by crypto_test
Run arc lint
Check that 2 errors are reported, on for name mismatch and one for
duplicated name.

Modify any cpp file directly located under src/
Run arc lint --trace
Check in the output that the file is not passed to the test linter.

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

Fabien created this revision.Feb 9 2019, 08:51
Herald added a reviewer: Restricted Project. · View Herald TranscriptFeb 9 2019, 08:51
Herald added a subscriber: schancel. · View Herald Transcript
deadalnix requested changes to this revision.Feb 9 2019, 16:32
deadalnix added inline comments.
109 ↗(On Diff #7274)

Indentation is wrong

118 ↗(On Diff #7274)

space after if

143 ↗(On Diff #7274)

Fix identation

This revision now requires changes to proceed.Feb 9 2019, 16:32
Fabien edited the summary of this revision. (Show Details)Feb 10 2019, 09:43
Fabien updated this revision to Diff 7285.Feb 10 2019, 09:51

Fix indentation and add space after if

deadalnix accepted this revision.Feb 10 2019, 13:37
This revision is now accepted and ready to land.Feb 10 2019, 13:37
This revision was automatically updated to reflect the committed changes.