Page MenuHomePhabricator

Add linter: Make sure we explicitly open all text files using UTF-8

Authored by Fabien on Jul 30 2019, 18:18.



...encoding in Python.

Backport of core PR13440 and PR13448

Depends on D3756 and D3759.

The linter has been converted to PHP and integrated into arcanist within
the same diff, as it is simple enough. It also gets improved to behave
correcty when the arguments are spead over multiple lines. It also graps
the open() call nested in another function call.

Test Plan
arc lint --everything

Find an open() occurrence in any python file and change the encoding
to encoding='foo'.

arc lint -- <the edited file>

Check the linter returns an error where the encoding has been changed.

Diff Detail

rABC Bitcoin ABC
Automatic diff as part of commit; lint not applicable.
Automatic diff as part of commit; unit tests not applicable.