fix(i18n): add missing French translations#2441
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
2 Skipped Deployments
|
Lunaria Status Overview🌕 This pull request will trigger status changes. Learn moreBy default, every PR changing files present in the Lunaria configuration's You can change this by adding one of the keywords present in the Tracked Files
Warnings reference
|
📝 WalkthroughWalkthroughThis pull request adds French language translations to the French locale file. The changes introduce six new i18n string entries: common boolean labels ( Suggested reviewers
🚥 Pre-merge checks | ✅ 1✅ Passed checks (1 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ 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)
i18n/locales/fr-FR.json (1)
946-946: Consider "Basculer" for consistency with other toggle actions.Whilst "Changer la largeur du conteneur" is understandable, other toggle actions in this file use "Basculer" (e.g., Line 941:
"toggle_tree": "Basculer l'arborescence", Line 619:"toggle_stack_mode": "Basculer le mode empilé"). For consistency, consider:♻️ Proposed refactor for consistency
- "toggle_container": "Changer la largeur du conteneur", + "toggle_container": "Basculer la largeur du conteneur",
ℹ️ Review info
⚙️ Run configuration
Configuration used: Organization UI
Review profile: CHILL
Plan: Pro
Run ID: c79ab8b0-bc4c-4a6b-92c2-392a51ae1060
📒 Files selected for processing (1)
i18n/locales/fr-FR.json
Codecov Report✅ All modified and coverable lines are covered by tests. 📢 Thoughts on this report? Let us know! |
Here, I chose "Changer" rather than "Basculer" because it seemed more natural in this context. This is also the verb used for the npmx.dev/i18n/locales/fr-FR.json Line 601 in 5a2b1a6 |
🔗 Linked issue
No linked issue
🧭 Context
French translations for the following keys are missing:
This PR adds translations for those keys.
📚 Description
French translations have been added for the keys listed above.