Complete Brazilian Portuguese translation - #7312
MukaSanches wants to merge 1 commit into
Conversation
|
Thank you for contributing to Thrive. Before your contribution can be accepted, you need to sign our CLA. Once your contribution has been accepted you can ask to be included in the credits (https://wiki.revolutionarygamesstudio.com/wiki/Team_Members) and you can apply to join the team and use this work as a sample: https://revolutionarygamesstudio.com/application/ Your pull request first will get automated checks run by our CI pipeline to detect problems. There may also be an AI code review. If there are problems found with these you should try to fix them or explain why the AI review was not correct. Once this initial review is complete a code reviewer from the Thrive team will take a look. Automated review happening first before the human review saves a lot of time for our reviewers. |
|
Important Review skippedReview was skipped due to path filters ⛔ Files ignored due to path filters (1)
CodeRabbit blocks several paths by default. You can override this behavior by explicitly including those paths in the path filters. For example, including ⚙️ Run configurationConfiguration used: Repository: Revolutionary-Games/Thrive/.coderabbit.yaml Review profile: CHILL Plan: Advanced Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
|
Thrive translations are typically handled in a different way. I also have to say that your PR description looks exactly like this is an AI-generated PR with LLM-generated translations, in which case we can't accept it into the game. |
|
Yep, we cannot accept any final text that is shown in the game that is LLM-generated. Also our general translation guidelines prohibit (pure) machine translations: https://wiki.revolutionarygamesstudio.com/wiki/Translation_Guidelines I can't speak Portuguese so I cannot validate the actual translation changes, but the PR description is extremely LLM-coded. Edit: I'll add this new info on the page as that guidelines page has not been updated since LLMs became widely used. |
Brief Description of What This PR Does
Completes the Brazilian Portuguese (pt_BR) translation for the current Thrive localization catalogue.
The update reviews untranslated and fuzzy entries with context-sensitive gameplay terminology, Thriveopedia content, scientific descriptions, later-stage documentation, organelle descriptions, and malformed Thrive compound markup.
Validation:
msgfmt --check --check-format locale/pt_BR.po -o /dev/nullpasseddotnet run --project Scripts -- translation-progressreportslocale/pt_BR.po: 100%dotnet run --project Scripts -- check compilepassedRelated Issues
No specific issue. This completes and reviews the existing Brazilian Portuguese localization.