ci: Add language mappings to crowdin.yml - #368
Conversation
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #368 +/- ##
=========================================
Coverage 46.42% 46.42%
Complexity 58 58
=========================================
Files 16 16
Lines 420 420
Branches 43 43
=========================================
Hits 195 195
Misses 197 197
Partials 28 28 ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
There was a problem hiding this comment.
Pull request overview
Adds Crowdin language code mappings in crowdin.yml to control how localized translation files are named when syncing translations into src/messages/.
Changes:
- Introduces
languages_mappingfor%two_letters_code%to map specific Crowdin locales (e.g.pt-BR,zh-CN) to custom output codes (e.g.pt_BR,zh_CN). - Adds mappings for several regional variants (Portuguese, Chinese, English, Dutch, Serbian, Kurdish).
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 76cacc8b9a
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| nl: nl_NL | ||
| zh-TW: zh_TW | ||
| sr-CS: sr_CS | ||
| sr: sr_RS | ||
| ku : kmr_TR |
There was a problem hiding this comment.
Keep base locales mapped to two-letter bundles
Do not remap base language codes (nl, sr, ku) to regional filenames here: this changes Crowdin export targets from messages_<lang>.properties to files like messages_nl_NL.properties/messages_sr_RS.properties/messages_kmr_TR.properties. In this repo, supported locales are still the base codes (see runtime-scripts/realm_settings_template.json), and message generation/lookup paths are built around messages_${code}.properties (see build-scripts/refresh_messages.mjs), so future Crowdin syncs will stop updating the files actually used for those locales.
Useful? React with 👍 / 👎.
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
What