Skip to content

Comments

feat(i18n): italian language update#1219

Merged
ghostdevv merged 1 commit intonpmx-dev:mainfrom
MatteoGabriele:feat/italian-languge-update
Feb 8, 2026
Merged

feat(i18n): italian language update#1219
ghostdevv merged 1 commit intonpmx-dev:mainfrom
MatteoGabriele:feat/italian-languge-update

Conversation

@MatteoGabriele
Copy link
Contributor

language update

@vercel
Copy link

vercel bot commented Feb 8, 2026

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

Project Deployment Actions Updated (UTC)
npmx.dev Ready Ready Preview, Comment Feb 8, 2026 3:43pm
2 Skipped Deployments
Project Deployment Actions Updated (UTC)
docs.npmx.dev Ignored Ignored Preview Feb 8, 2026 3:43pm
npmx-lunaria Ignored Ignored Feb 8, 2026 3:43pm

Request Review

@github-actions
Copy link

github-actions bot commented Feb 8, 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
lunaria/files/it-IT.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

codecov bot commented Feb 8, 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!

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Feb 8, 2026

📝 Walkthrough

Walkthrough

This pull request adds Italian translation keys to locale files. The changes introduce a rate_limited message for npm rate-limit scenarios, a size localisation group with keys for bytes, kilobytes, and megabytes, trends section display helpers, authentication modal input error text, and a totalLikes metric entry under facets. The modifications span two locale files with a total of 19 lines added and 2 removed.

Possibly related PRs

Suggested reviewers

  • danielroe
🚥 Pre-merge checks | ✅ 1
✅ Passed checks (1 passed)
Check name Status Explanation
Description check ✅ Passed The pull request description is directly related to the changeset, which updates Italian language translations (i18n) in two files.

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

✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

No actionable comments were generated in the recent review. 🎉

🧹 Recent nitpick comments
i18n/locales/it-IT.json (1)

28-28: Keep it-IT as a delta-only locale (if it is a variant).

This file currently contains the full Italian corpus. In this repo, country-variant locales should only include keys that differ from the base language file. Consider moving shared keys into the base Italian file and keeping only overrides here; if this is meant to be the base locale, rename accordingly and update the i18n config.
Based on learnings: "In the npmx.dev project, when using country locale variants (e.g., es-419, es-ES), place only translations that differ from the base language in variant JSON files (e.g., es-419.json). The base file (es.json) is loaded first, then the variant file overlays any keys it defines. This behavior is provided by vue-i18n's multiple-files feature. See CONTRIBUTING.md under 'Country variants (advanced)' for details."


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

@ghostdevv ghostdevv added the i18n Related to internationalization label Feb 8, 2026
@ghostdevv ghostdevv added this pull request to the merge queue Feb 8, 2026
Merged via the queue into npmx-dev:main with commit 840f3d4 Feb 8, 2026
17 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

i18n Related to internationalization

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants