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
F14470663: D21776.id51932.diff
Fri, Dec 27, 1:23 PM
F14468388: D21776.id51913.diff
Fri, Dec 27, 12:24 PM
Unknown Object (File)
Thu, Dec 26, 8:50 PM
Unknown Object (File)
Sat, Dec 21, 1:07 PM
Unknown Object (File)
Tue, Dec 17, 9:30 AM
Unknown Object (File)
Thu, Dec 5, 10:20 PM
Unknown Object (File)
Thu, Dec 5, 4:01 AM
Unknown Object (File)
Mon, Dec 2, 10:44 AM
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.