feat(i18n): update pt-BR locale#2625
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
2 Skipped Deployments
|
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (1)
📝 WalkthroughSummary by CodeRabbit
WalkthroughAdds and expands multiple Portuguese (Brazilian) translation keys in Changes
Possibly related PRs
Suggested reviewers
🚥 Pre-merge checks | ✅ 4✅ Passed checks (4 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 |
|
Hello! Thank you for opening your first PR to npmx, @joaopalmeiro! 🚀 Here’s what will happen next:
|
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
|
Codecov Report✅ All modified and coverable lines are covered by tests. 📢 Thoughts on this report? Let us know! |
There was a problem hiding this comment.
Actionable comments posted: 2
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Inline comments:
In `@i18n/locales/pt-BR.json`:
- Around line 1613-1616: Update the Portuguese date string to use
day-before-month and lowercase month for natural pt-BR phrasing: replace the
"dates" value currently "Fevereiro 14 – 21" with "14 – 21 de fevereiro" and keep
the surrounding keys ("p1", "p2", "garden") unchanged so the sentence reads
naturally in pt-BR.
- Around line 24-26: The translations for the footer keys "resources",
"features", and "other" in pt-BR.json use Title Case ("Recursos",
"Funcionalidades", "Outros") but the rest of the footer.* entries are lowercase;
change these three values to lowercase ("recursos", "funcionalidades", "outros")
so they match the existing footer styling and remain consistent with other
footer keys.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Path: .coderabbit.yaml
Review profile: CHILL
Plan: Pro
Run ID: 15daf2c1-c71c-4bab-91e4-4459d2b3fc55
📒 Files selected for processing (1)
i18n/locales/pt-BR.json
| "resources": "Recursos", | ||
| "features": "Funcionalidades", | ||
| "other": "Outros" |
There was a problem hiding this comment.
Capitalisation inconsistent with the rest of the footer section.
All pre-existing footer.* values are lowercase (e.g. "sobre", "documentação", "código-fonte", "redes sociais", "atalhos de teclado", "marca"), but the three new entries use Title Case. If the English source uses Title Case here intentionally (matching e.g. CSS text-transform), this is fine; otherwise it will render visibly inconsistent in the footer UI.
✏️ Suggested fix to align with the existing footer style
"brand": "marca",
- "resources": "Recursos",
- "features": "Funcionalidades",
- "other": "Outros"
+ "resources": "recursos",
+ "features": "funcionalidades",
+ "other": "outros"
},📝 Committable suggestion
‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.
| "resources": "Recursos", | |
| "features": "Funcionalidades", | |
| "other": "Outros" | |
| "brand": "marca", | |
| "resources": "recursos", | |
| "features": "funcionalidades", | |
| "other": "outros" |
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.
In `@i18n/locales/pt-BR.json` around lines 24 - 26, The translations for the
footer keys "resources", "features", and "other" in pt-BR.json use Title Case
("Recursos", "Funcionalidades", "Outros") but the rest of the footer.* entries
are lowercase; change these three values to lowercase ("recursos",
"funcionalidades", "outros") so they match the existing footer styling and
remain consistent with other footer keys.
There was a problem hiding this comment.
Title Case for consistency purposes:
Lines 14 to 27 in ebcfc01
There was a problem hiding this comment.
Only users with a collaborator, contributor, member, or owner role can interact with CodeRabbit.
|
Thanks for your first contribution, @joaopalmeiro! ✨ We'd love to welcome you to the npmx community. Come and say hi on Discord! And once you've joined, visit npmx.wamellow.com to claim the contributor role. |
Hi! 👋
🧭 Context
There are still some untranslated strings in the pt-BR locale (
en.jsonvs.pt-BR.json).📚 Description
This PR adds the missing translations to the pt-BR locale. Two strings were also cleaned of invisible characters.
References