- Group Reviewers
- rARCc450fa6c069e: `pht`ize a bunch of strings
- rARC Arcanist
No Test Coverage
- Build Status
Buildable 4049 Build 4062: [Placeholder Plan] Wait for 30 Seconds
One technical error.
Missing a parameter.
Well spotted, I though that the linter was meant to pick this kind of stuff up.
A sprintf() linter would be helpful, but we don't have one currently.
We have checks that the parameter is a string, but we don't try to parse the pattern.
(It gets hard with %3$s and stuff, but pht(), qsprintf(), etc., don't support that stuff anyway.)