Page MenuHomePhabricator

Add some missing translations
ClosedPublic

Authored by joshuaspence on Nov 2 2015, 10:28 AM.
Tags
None
Referenced Files
Unknown Object (File)
Sun, Mar 17, 8:41 PM
Unknown Object (File)
Sun, Mar 17, 8:33 PM
Unknown Object (File)
Sun, Mar 17, 7:27 PM
Unknown Object (File)
Sun, Mar 17, 6:33 PM
Unknown Object (File)
Sun, Mar 17, 5:49 PM
Unknown Object (File)
Sat, Mar 16, 6:23 PM
Unknown Object (File)
Thu, Mar 14, 4:51 PM
Unknown Object (File)
Mon, Mar 11, 2:04 AM
Subscribers

Details

Summary

Add some missing translations from D14071.

Test Plan

I'm actually not sure where these are meant to go... we don't seem to store any translations in rPHU.

Diff Detail

Repository
rPHU libphutil
Branch
master
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 8546
Build 9863: Run Core Tests
Build 9862: arc lint + arc unit

Event Timeline

joshuaspence retitled this revision from to Add some missing translations.
joshuaspence updated this object.
joshuaspence edited the test plan for this revision. (Show Details)
joshuaspence added a reviewer: epriestley.
epriestley edited edge metadata.

These should go in libphutil/, we just haven't translated anything there yet. Just add a similar translation file, like the ones in Arcanist and Phabricator.

src/infrastructure/internationalization/translation/PhabricatorUSEnglishTranslation.php
1435–1436 ↗(On Diff #34722)

Consider spelling out the singular cases, "Analyzing 1 file with 1 subprocess..." is still more stilted in English than, e.g., "Analyzing one file with a single subprocess..."

1445 ↗(On Diff #34722)

Missing closing parenthesis.

This revision is now accepted and ready to land.Nov 2 2015, 1:02 PM
joshuaspence marked 2 inline comments as done.
joshuaspence edited edge metadata.
  • Move to rPHU
  • Fix missing parentheses
This revision was automatically updated to reflect the committed changes.