Page MenuHomePhabricator

`pht`ize a bunch of strings
ClosedPublic

Authored by joshuaspence on Jan 26 2015, 5:01 AM.
Tags
None
Referenced Files
F15566259: D11501.id27660.diff
Thu, May 1, 5:39 PM
F15554548: D11501.diff
Mon, Apr 28, 7:34 AM
F15506952: D11501.diff
Tue, Apr 15, 3:20 PM
F15477828: D11501.id27675.diff
Apr 7 2025, 6:04 PM
F15471545: D11501.diff
Apr 5 2025, 5:52 AM
F15467754: D11501.id27660.diff
Apr 3 2025, 5:45 PM
F15467042: D11501.id.diff
Apr 3 2025, 12:05 PM
F15463846: D11501.diff
Apr 2 2025, 4:02 AM
Subscribers

Details

Summary

Self-explanatory.

Test Plan

Eyeball it.

Diff Detail

Repository
rARC Arcanist
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

joshuaspence retitled this revision from to `pht`ize a bunch of strings.
joshuaspence updated this object.
joshuaspence edited the test plan for this revision. (Show Details)
joshuaspence added a reviewer: epriestley.
epriestley edited edge metadata.

One technical error.

src/lint/engine/ArcanistConfigurationDrivenLintEngine.php
43

Missing a parameter.

This revision is now accepted and ready to land.Jan 26 2015, 3:15 PM
src/lint/engine/ArcanistConfigurationDrivenLintEngine.php
43

Well spotted, I though that the linter was meant to pick this kind of stuff up.

This revision was automatically updated to reflect the committed changes.

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.)