Skip to content

Conversation

@sozua
Copy link
Contributor

@sozua sozua commented Dec 8, 2025

Description

The pt-br (Brazilian Portuguese) language is now translated and reviewed in Crowdin, so I think it's time to enable it in the website.

Validation

Check in the language dropdown that "Brazilian Portuguese" is visible and navigates to /pt-br pages correctly.

image

Related Issues

N/A

Check List

  • I have read the Contributing Guidelines and made commit messages that follow the guideline.
  • I have run pnpm format to ensure the code follows the style guide.
  • I have run pnpm test to check if all tests are passing.
  • I have run pnpm build to check if the website builds without errors.
  • I've covered new added functionality with unit tests if necessary.

@sozua sozua requested a review from a team as a code owner December 8, 2025 17:22
Copilot AI review requested due to automatic review settings December 8, 2025 17:22
@vercel
Copy link

vercel bot commented Dec 8, 2025

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

Project Deployment Preview Updated (UTC)
nodejs-org Ready Ready Preview Dec 8, 2025 5:23pm

@github-actions
Copy link
Contributor

github-actions bot commented Dec 8, 2025

👋 Codeowner Review Request

The following codeowners have been identified for the changed files:

Team reviewers: @nodejs/nodejs-website

Please review the changes when you have a chance. Thank you! 🙏

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR enables Brazilian Portuguese (pt-br) translation on the Node.js website by updating the i18n configuration to set enabled: true and adding the hrefLang value. The translation files are already complete in Crowdin and ready for use.

  • Sets Brazilian Portuguese as an enabled language in the i18n configuration
  • Adds hrefLang value to enable proper HTML lang attribute generation

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copy link
Contributor

@bmuenzenmeyer bmuenzenmeyer left a comment

Choose a reason for hiding this comment

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

congrats!

Image

@codecov
Copy link

codecov bot commented Dec 8, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 73.71%. Comparing base (aaf3ca0) to head (ca4d4be).
⚠️ Report is 3 commits behind head on main.
✅ All tests successful. No failed tests found.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #8403      +/-   ##
==========================================
- Coverage   73.73%   73.71%   -0.02%     
==========================================
  Files         107      107              
  Lines        9155     9155              
  Branches      308      308              
==========================================
- Hits         6750     6749       -1     
- Misses       2403     2404       +1     
  Partials        2        2              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@ovflowd ovflowd added this pull request to the merge queue Dec 9, 2025
@ovflowd ovflowd added the fast-track Fast Tracking PRs label Dec 9, 2025
@ovflowd
Copy link
Member

ovflowd commented Dec 9, 2025

I've fast-tracked as I don't see reasons for why not getting this shipped asap :)

Merged via the queue into nodejs:main with commit fb4deb8 Dec 9, 2025
23 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

fast-track Fast Tracking PRs

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants