Skip to content

feat(i18n): update pt-BR locale#2625

Merged
graphieros merged 2 commits intonpmx-dev:mainfrom
joaopalmeiro:feat/update-pt-br
Apr 26, 2026
Merged

feat(i18n): update pt-BR locale#2625
graphieros merged 2 commits intonpmx-dev:mainfrom
joaopalmeiro:feat/update-pt-br

Conversation

@joaopalmeiro
Copy link
Copy Markdown
Contributor

@joaopalmeiro joaopalmeiro commented Apr 25, 2026

Hi! 👋

🧭 Context

There are still some untranslated strings in the pt-BR locale (en.json vs. pt-BR.json).

📚 Description

This PR adds the missing translations to the pt-BR locale. Two strings were also cleaned of invisible characters.

References

@vercel
Copy link
Copy Markdown

vercel Bot commented Apr 25, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
npmx.dev Ready Ready Preview, Comment Apr 25, 2026 7:31pm
2 Skipped Deployments
Project Deployment Actions Updated (UTC)
docs.npmx.dev Ignored Ignored Preview Apr 25, 2026 7:31pm
npmx-lunaria Ignored Ignored Apr 25, 2026 7:31pm

Request Review

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented Apr 25, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: c3e80dcc-2d20-4d93-ac42-1c66aed35bc9

📥 Commits

Reviewing files that changed from the base of the PR and between 854fa1f and 1edb45b.

📒 Files selected for processing (1)
  • i18n/locales/pt-BR.json

📝 Walkthrough

Summary by CodeRabbit

  • New Features

    • Introduced new theme colour options, including sky accent and expanded background colour palette (stone, zinc, slate)
  • Chores

    • Expanded Portuguese (Brazil) language localisations across the application, including footer navigation links, keyboard shortcuts, package management features, timeline labels and messaging, code viewer interface, comparison chart analytics, vacations page content with additional stats, and brand asset guidelines

Walkthrough

Adds and expands multiple Portuguese (Brazilian) translation keys in i18n/locales/pt-BR.json covering footer navigation, shortcuts, theme options, package/timeline/README messaging, code viewer, compare chart labels, vacations content, and brand/logo strings.

Changes

Cohort / File(s) Summary
Locale file
i18n/locales/pt-BR.json
Added footer labels (resources, features, other); shortcuts.open_timeline; theme options (settings.theme.accent_colors.sky, settings.theme.background_themes.stone/zinc/slate); package keys (size_decrease, replacement.example, replacement.native_no_version, skills_cli, links.timeline, readme.error_loading); new comprehensive package.timeline subtree; code.open_path_dropdown; compare updates (scatter_chart.freshness_score, scatter_chart.copy_alt.description, scatter_chart.filename); vacations additions (illustration_alt, poke_log, what.p1/dates/p2/garden, stats.commits); brand logos and guidelines (brand.logos.*, brand.guidelines.message, brand.guidelines.discord_link_text). Lines: +67/-10.

Possibly related PRs

Suggested reviewers

  • graphieros
  • ghostdevv
🚥 Pre-merge checks | ✅ 4
✅ Passed checks (4 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly summarizes the main change: adding missing translations to the Portuguese-Brazilian (pt-BR) locale file.
Description check ✅ Passed The description is directly related to the changeset, providing context about untranslated strings in pt-BR locale and explaining what the PR accomplishes.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

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.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@github-actions
Copy link
Copy Markdown

Hello! Thank you for opening your first PR to npmx, @joaopalmeiro! 🚀

Here’s what will happen next:

  1. Our GitHub bots will run to check your changes.
    If they spot any issues you will see some error messages on this PR.
    Don’t hesitate to ask any questions if you’re not sure what these mean!

  2. In a few minutes, you’ll be able to see a preview of your changes on Vercel

  3. One or more of our maintainers will take a look and may ask you to make changes.
    We try to be responsive, but don’t worry if this takes a few days.

@github-actions
Copy link
Copy Markdown

github-actions Bot commented Apr 25, 2026

Lunaria Status Overview

🌕 This pull request will trigger status changes.

Learn more

By default, every PR changing files present in the Lunaria configuration's files property will be considered and trigger status changes accordingly.

You can change this by adding one of the keywords present in the ignoreKeywords property in your Lunaria configuration file in the PR's title (ignoring all files) or by including a tracker directive in the merged commit's description.

Tracked Files

File Note
i18n/locales/pt-BR.json Localization changed, will be marked as complete.
Warnings reference
Icon Description
🔄️ The source for this localization has been updated since the creation of this pull request, make sure all changes in the source have been applied.

@codecov
Copy link
Copy Markdown

codecov Bot commented Apr 25, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ All tests successful. No failed tests found.

📢 Thoughts on this report? Let us know!

Copy link
Copy Markdown
Contributor

@coderabbitai coderabbitai Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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

📥 Commits

Reviewing files that changed from the base of the PR and between ebcfc01 and 854fa1f.

📒 Files selected for processing (1)
  • i18n/locales/pt-BR.json

Comment thread i18n/locales/pt-BR.json
Comment on lines +24 to +26
"resources": "Recursos",
"features": "Funcionalidades",
"other": "Outros"
Copy link
Copy Markdown
Contributor

@coderabbitai coderabbitai Bot Apr 25, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

⚠️ Potential issue | 🟡 Minor

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.

Suggested change
"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.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Title Case for consistency purposes:

"footer": {
"about": "about",
"blog": "blog",
"docs": "docs",
"source": "source",
"social": "social",
"chat": "chat",
"builders_chat": "builders",
"keyboard_shortcuts": "keyboard shortcuts",
"brand": "brand",
"resources": "Resources",
"features": "Features",
"other": "Other"
},

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Only users with a collaborator, contributor, member, or owner role can interact with CodeRabbit.

Comment thread i18n/locales/pt-BR.json
Copy link
Copy Markdown
Contributor

@graphieros graphieros left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you :)

@graphieros graphieros added this pull request to the merge queue Apr 26, 2026
Merged via the queue into npmx-dev:main with commit 4760dca Apr 26, 2026
21 checks passed
@github-actions
Copy link
Copy Markdown

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.

@joaopalmeiro joaopalmeiro deleted the feat/update-pt-br branch April 26, 2026 19:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants