-
Notifications
You must be signed in to change notification settings - Fork 786
mls (v4.15) #3019
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: v29
Are you sure you want to change the base?
mls (v4.15) #3019
Conversation
|
|
WalkthroughFrench localization expanded with new UI keys and renames of existing entries. New comprehensive Indonesian language file added with 953 translation strings. Language metadata updated to register Indonesian. Minor Montreal translation adjustment in Turkish file. Changes
Estimated code review effort🎯 3 (Moderate) | ⏱️ ~25 minutes Poem
🚥 Pre-merge checks | ✅ 2 | ❌ 1❌ Failed checks (1 inconclusive)
✅ Passed checks (2 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. 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 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actionable comments posted: 3
Caution
Some comments are outside the diff and can’t be posted inline due to platform limitations.
⚠️ Outside diff range comments (1)
resources/lang/fr.json (1)
458-458: Fix small text issues in French strings.One string shows a literal quote, one has a trailing space, and one word is misspelled.
🛠️ Proposed fixes
- "performance_overlay_desc": "Activer/désactiver la surcouche de performances. Lorsqu'elle est activée, la surcouche de performances sera affichée. Appuyez sur Maj+D pendant le jeu pour l\"activer.", + "performance_overlay_desc": "Activer/désactiver la surcouche de performances. Lorsqu'elle est activée, la surcouche de performances sera affichée. Appuyez sur Maj+D pendant le jeu pour l'activer.", @@ - "menu_shortcuts": "Menu des raccourcis ", + "menu_shortcuts": "Menu des raccourcis", @@ - "building": "Batîment", + "building": "Bâtiment",Also applies to: 488-488, 883-883
🤖 Fix all issues with AI agents
In `@resources/lang/id.json`:
- Line 313: The JSON value for the translation key "disabled_units" contains a
typo ("Nonaktfikan Units"); update the string to correct Indonesian (e.g., "Unit
Nonaktif" or "Nonaktifkan Unit" as appropriate for the UI) and then scan the
rest of resources/lang/id.json for similar spelling mistakes and fix the other
user-facing strings referenced in the review (the other Indonesian translation
keys around the listed sections) so all phrases are valid Indonesian and
consistent in terminology.
- Line 286: The translation entry for the key "surrounded" has a typo in its
value ("Surrourded"); update the value to the correct spelling "Surrounded" in
the "surrounded" entry in resources/lang/id.json so it matches the source
en.json.
In `@resources/lang/tr.json`:
- Line 273: The "montreal" translation value contains a trailing newline
("Montreal\n") which causes an unwanted line break; update the "montreal" key's
value to remove the "\n" so it reads "Montreal" (edit the JSON entry for the
"montreal" property).
Co-authored-by: coderabbitai[bot] <136622811+coderabbitai[bot]@users.noreply.github.com>
Description:
mls for v29
Version identifier within MLS: 4.14
This is the last mls for v29.
Please complete the following:
Please put your Discord username so you can be contacted if a bug or regression is found:
aotumuri