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
Lint
Lint Not Applicable
Unit
Tests Not Applicable

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.