Page MenuHomePhabricator

[LINTER] Display the line number in the lint-locale-dependence.sh linter
ClosedPublic

Authored by Fabien on Mar 19 2019, 08:23.

Details

Summary

This makes the output to look like this:

<source file>:<line nunmber>:<code snippet>

Having the line number makes it easier to locate the code line and make
sure all the issues are fixed. This is especially true in this case
where it is likely to have multiple times the same line in the file.

Depends on D2698

Test Plan
./test/lint/lint-locale-dependence.sh

Check the script outputs not error (no output)

Comment any of the known exceptions in the lint-locale-dependence.sh
script.

./test/lint/lint-locale-dependence.sh

Check the output contains the line number

Diff Detail

Repository
rABC Bitcoin ABC
Branch
linter_local_line_number
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 5244
Build 8551: Bitcoin ABC Buildbot (legacy)
Build 8550: arc lint + arc unit

Event Timeline

This revision is now accepted and ready to land.Mar 22 2019, 21:47
This revision was automatically updated to reflect the committed changes.