Page MenuHomePhabricator

Catch more product names in "pht()", and replace newly matched Arcanist product names
ClosedPublic

Authored by epriestley on Apr 25 2022, 8:04 PM.
Tags
None
Referenced Files
F18770911: D21776.diff
Wed, Oct 8, 4:08 PM
F18756451: D21776.diff
Sun, Oct 5, 11:29 AM
F18750111: D21776.id51932.diff
Sat, Oct 4, 6:32 AM
F18699795: D21776.id.diff
Sat, Sep 27, 1:43 PM
F18653697: D21776.id51913.diff
Sun, Sep 21, 3:24 PM
F18616613: D21776.diff
Sep 14 2025, 6:19 PM
F18349965: D21776.diff
Aug 26 2025, 7:01 PM
F18232940: D21776.id.diff
Aug 20 2025, 4:09 PM
Subscribers
None

Details

Summary

Ref T13658. The lint rule called "getStringLiteralValue()", which produces string literals for fewer nodes than "evalStatic()".

Switch to "evalStatic()", then fix new warnings.

Test Plan

This test plan is non-exhaustive.

  • Ran "arc lint --everything --output summary" to generate new warnings.

Diff Detail

Repository
rARC Arcanist
Branch
product11
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 25669
Build 35497: arc lint + arc unit

Event Timeline

epriestley created this revision.
This revision was not accepted when it landed; it landed in state Needs Review.Apr 25 2022, 11:46 PM
This revision was automatically updated to reflect the committed changes.