[Proof of concept] Add script to generate locale files, and add PT as locale option. #20330
Conversation
This file contains 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
Uffizzi Ephemeral Environment
|
|
If I may ask, what is the point of this? You only need translations once. Generating all the locales in another language, even with AI, will rarely be accurate. There is a good community trying to provide translations that are accurate per language and per region: was this PT locale Portuguese or Brazilian or any other variant? I tried myself in the past by doing it with Spanish from Spain but my requests in Github and in Forem.dev were ignored. Simply nurturing and taking a little care of the community will provide good translations that will cover the most used languages very rapidly. Just my two cents. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
What type of PR is this? (check all applicable)
Description
We have had a chicken/egg problem of not necessarily having a line-of-site towards being able to fully complete and make functional use of our locale setting due to the maintenance burden of multiple locales.
This PR adds a bin script to automatically generate locale files, always using
/../../en.ymlas the source of truth.Unlike other auto-translators, this asks GPT-4 to evaluate the full context of the file and make intelligent, context-rich decisions about which terms to prefer.
This could be improved over time in a variety of ways — including adding more comments to the
/../../en.ymlfiles to provide more assurance that the script will understand the context and make the appropriate choice.This PR updated our
fr.ymlfiles and addedpt.yml, as Portuguese visitors are some of the more popular visitors of the platform. If this POC works, we can use this as a forcing function to finish localization functionality and make it usable across many languages.Note: There are a few times where the yml breaks and I reverted to the original. For this to work ideally, this should be caught as part of the script itself, but for now it is not.