feat: Add Czech translation (cs) - #144
Conversation
📝 WalkthroughWalkthroughCzech ( Czech Locale Addition
Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes Possibly related PRs
Suggested labels
Suggested reviewers
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
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.
🧹 Nitpick comments (1)
messages/cs.json (1)
816-817: 📐 Maintainability & Code Quality | 🔵 Trivial | ⚡ Quick winFix inconsistent ICU plural style.
Lines 816–817 use the
one/few/othercategories, while the rest of this file uses explicit=0/=1/few/other. Standardize on one style for maintainability. Prefer the=0/=1/few/otherpattern already used elsewhere in this file since it handles Czech's complex plural rules more explicitly.♻️ Proposed fix
- "deleteSelectedLogsDescription": "Smazat {count, plural, one {# vybraný protokol} few {# vybrané protokoly} other {# vybraných protokolů}}. Tuto akci nelze vrátit zpět.", - "willDeleteSelectedLogs": "{count, plural, one {# protokol} few {# protokoly} other {# protokolů}} bude trvale smazáno" + "deleteSelectedLogsDescription": "Smazat {count, plural, =0 {vybrané protokoly} =1 {1 vybraný protokol} few {# vybrané protokoly} other {# vybraných protokolů}}. Tuto akci nelze vrátit zpět.", + "willDeleteSelectedLogs": "{count, plural, =0 {Žádné protokoly} =1 {1 protokol} few {# protokoly} other {# protokolů}} bude trvale smazáno"🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the rest with a brief reason, keep changes minimal, and validate. In `@messages/cs.json` around lines 816 - 817, The ICU plural forms in the `deleteSelectedLogsDescription` and `willDeleteSelectedLogs` messages are inconsistent with the rest of `messages/cs.json`; update these entries to use the same explicit `=0`/`=1`/`few`/`other` pattern used elsewhere in the file. Make the change in the two affected message keys so the Czech plural handling stays uniform and easy to maintain.
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Nitpick comments:
In `@messages/cs.json`:
- Around line 816-817: The ICU plural forms in the
`deleteSelectedLogsDescription` and `willDeleteSelectedLogs` messages are
inconsistent with the rest of `messages/cs.json`; update these entries to use
the same explicit `=0`/`=1`/`few`/`other` pattern used elsewhere in the file.
Make the change in the two affected message keys so the Czech plural handling
stays uniform and easy to maintain.
ℹ️ Review info
⚙️ Run configuration
Configuration used: defaults
Review profile: CHILL
Plan: Pro
Run ID: 14f59b7c-42e4-40c5-8951-16d6dccdbb80
📒 Files selected for processing (8)
README.mdlib/locales.tsmessages/cs.jsonmessages/de.jsonmessages/en.jsonmessages/es.jsonmessages/fr.jsonmessages/it.json
Hello, awesome project!!
I tried to add the Czech language without any further intervention.
I was based on commit 0fa7e8d feat: Add French translation (fr) (#139) and ddf251e (Spanish)
Summary by CodeRabbit