Page MenuHomePhabricator

Add a "product name literal in pht()" linter
ClosedPublic

Authored by epriestley on Apr 25 2022, 4:41 PM.
Tags
None
Referenced Files
F18764255: D21763.id51882.diff
Tue, Oct 7, 5:07 AM
F18755960: D21763.diff
Sun, Oct 5, 8:22 AM
F18751732: D21763.id.diff
Sat, Oct 4, 12:23 PM
F18737847: D21763.diff
Wed, Oct 1, 12:58 PM
F18702819: D21763.diff
Sun, Sep 28, 1:43 AM
F18657302: D21763.id51899.diff
Tue, Sep 23, 1:17 AM
F18651054: D21763.diff
Sun, Sep 21, 2:47 AM
F18625722: D21763.id51899.diff
Mon, Sep 15, 11:24 PM
Subscribers
None

Details

Summary

Ref T13658. One challenge in forking Phabricator is product name references in user-visible strings, particularly in "pht()".

Add a linter to identify the use of product name literals in "pht()" text.

The linter could fix these automatically, but it looks like there are fewer than 200 across both Arcanist and Phabricator and some sampling suggests that many are probably clearer when rewritten into generic language anyway.

Test Plan

Ran linter, saw it pop out reasonable warnings.

Diff Detail

Repository
rARC Arcanist
Lint
Lint Not Applicable
Unit
Tests Not Applicable