Add European Portuguese (pt-PT) as the 29th translation locale - #657
Merged
Conversation
Completes the customer's 29-language request. The customer's "Portuguese (PT)" is European Portuguese (Portugal), which is distinct from the Brazilian Portuguese (pt-BR) already shipped, so pt-PT is added alongside pt-BR (29 locales total). - LOCALE_NAMES + the workflow DEFAULT_LOCALES / dispatch default now include pt-PT. - disclaimers.json: European Portuguese machine-translation disclaimer added. - auto-translations/pt-PT/: full translation of all playbooks + shared dependencies (78 files), each with the machine-translation disclaimer. - call_model now also retries ConnectionError (reset/aborted/refused); an unretried connection reset had crashed a translation run mid-pass. - README: mark the dedicated self-hosted 'translation' runner and the LLM_* repository secrets as Implemented (now live), present-tense wording. English playbooks/ are unchanged.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Adds European Portuguese (
pt-PT) as the 29th locale, completing the customer's 29-language request. The customer's "Portuguese (PT)" is European Portuguese (Portugal), which is a distinct variant from the Brazilian Portuguese (pt-BR) already shipped, sopt-PTis added alongsidept-BR(29 locales total).What's included
pt-PTlocale wiring: added toLOCALE_NAMESintranslate_playbook.pyand toDEFAULT_LOCALES(plus the manual-dispatch default) intranslate-playbooks.yml.disclaimers.json: European Portuguese machine-translation disclaimer added (29 locales).auto-translations/pt-PT/: full translation of all playbooks + shared dependencies (78 files), each with the machine-translation> [!WARNING]disclaimer at the top. Matchespt-BRfile-for-file (78 each).call_modelnow also retriesConnectionError(reset/aborted/refused) — an unretried connection reset had crashed a run mid-pass.auto-translations/README.mdnow marks the dedicated self-hostedtranslationrunner and theLLM_*repository secrets as Implemented (they are now live), with present-tense wording.Notes
playbooks/is unchanged.pt-PT= 78 files, 0 missing, 0 orphans, disclaimer present.