Skip to content

fix(i18n): update French localizations#2560

Merged
graphieros merged 8 commits intonpmx-dev:mainfrom
WarningImHack3r:baguette-3
Apr 18, 2026
Merged

fix(i18n): update French localizations#2560
graphieros merged 8 commits intonpmx-dev:mainfrom
WarningImHack3r:baguette-3

Conversation

@WarningImHack3r
Copy link
Copy Markdown
Contributor

@WarningImHack3r WarningImHack3r commented Apr 17, 2026

🔗 Linked issue

🧭 Context

Some new French localizations were missing.

This PR adds them and corrects existing AI-generated translations that were incorrect.

📚 Description

@vercel
Copy link
Copy Markdown

vercel bot commented Apr 17, 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 18, 2026 7:55am
2 Skipped Deployments
Project Deployment Actions Updated (UTC)
docs.npmx.dev Ignored Ignored Preview Apr 18, 2026 7:55am
npmx-lunaria Ignored Ignored Apr 18, 2026 7:55am

Request Review

@github-actions
Copy link
Copy Markdown

github-actions bot commented Apr 17, 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/fr-FR.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.

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai bot commented Apr 17, 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: 95c7d985-1793-4e3f-af1e-fc6198b7ed48

📥 Commits

Reviewing files that changed from the base of the PR and between 8ae49df and 626bc03.

📒 Files selected for processing (1)
  • i18n/locales/fr-FR.json
✅ Files skipped from review due to trivial changes (1)
  • i18n/locales/fr-FR.json

📝 Walkthrough

Summary by CodeRabbit

  • Localisation
    • Enhanced French translations: added shortcut and timeline navigation labels, new vacations section, and timeline/pagination/event strings for versions
    • Improved package UI copy: README error messaging, skills text and maintainer avatar template
    • Refined comparison text: moved facet selection labels, added chart no-data message
    • Updated brand/logo headings and typography guidance

Walkthrough

Updated the French locale file i18n/locales/fr-FR.json: added new translation keys and subtrees, restructured several package.versions and compare keys, removed obsolete compare.trends keys, and revised multiple brand/maintainer/skill-related strings. Changes are localization-only.

Changes

Cohort / File(s) Summary
French localisation — additions
i18n/locales/fr-FR.json
Added keys/subtrees: shortcuts.open_timeline, package.links.timeline, package.readme.error_loading, package.versions.timeline.* (timeline loading/error, pagination, event labels), top-level vacations.*, and compare.no_chartable_data. Added compare.facets.* selection/deselection/summary keys.
French localisation — reorganisations & edits
i18n/locales/fr-FR.json
Moved package.versions.page_title, package.versions.current_tags, package.versions.no_match_filter from package.versions.copy_alt.*package.versions.*. Removed duplicate selection keys under compare.trends.*. Updated values for package.skills.skills_cli, package.maintainers.maintainer_template (adds {avatar} marker), and multiple logo_menu.browse_brand / brand.* entries (headings, download labels, typography/guidelines).
🚥 Pre-merge checks | ✅ 2
✅ Passed checks (2 passed)
Check name Status Explanation
Title check ✅ Passed The title 'fix(i18n): update French localizations' accurately summarizes the main change: fixing and updating French translation strings.
Description check ✅ Passed The description is directly related to the changeset, explaining that missing French localizations were added and incorrect AI-generated translations were corrected.

✏️ 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.

@codecov
Copy link
Copy Markdown

codecov bot commented Apr 17, 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/fr-FR.json`:
- Line 1609: Fix the typo in the French locale string for key "p1": replace the
incorrect word "main" with "mais" inside the value for "p1" so the sentence
reads "...mais tout le monde a principalement passé du temps..."; update the
JSON value for "p1" accordingly.
- Around line 569-570: The keys dep_increase and dep_decrease need vue-i18n
plural forms so count=1 renders correctly; update the translation values for
dep_increase and dep_decrease to use the singular|plural pipe format (provide
the singular form "1 dépendance ajoutée"/"1 dépendance enlevée" before the | and
the plural "{count} dépendances ajoutées"/"{count} dépendances enlevées" after)
so the lookup in app/pages/package-timeline/[[org]]/[packageName].vue outputs
proper singular/plural text.
🪄 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: 7855c281-3fa1-4112-bb5f-d3b1f9b794d1

📥 Commits

Reviewing files that changed from the base of the PR and between cbcdc54 and 9d63a75.

📒 Files selected for processing (1)
  • i18n/locales/fr-FR.json

Comment thread i18n/locales/fr-FR.json Outdated
Comment thread i18n/locales/fr-FR.json Outdated
WarningImHack3r and others added 2 commits April 17, 2026 19:32
Co-authored-by: coderabbitai[bot] <136622811+coderabbitai[bot]@users.noreply.github.com>
Co-authored-by: coderabbitai[bot] <136622811+coderabbitai[bot]@users.noreply.github.com>
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: 4

🤖 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/fr-FR.json`:
- Line 426: The translation key "skills_cli" uses the English word "skills"
("CLI de skills") which is inconsistent with the rest of the section that uses
"compétences"; update the value for the "skills_cli" key to use the French term,
e.g., "CLI de compétences", so it matches other keys like "title",
"skills_available", and "available_skills".
- Line 1595: Update the French translation value for the "meta_description" key
in fr-FR.json to use the standard past participle: replace "L'équipe npmx se
rechargeait. Notre discord a réouvert au bout d'une semaine." with the corrected
wording "L'équipe npmx se rechargeait. Notre discord a rouvert au bout d'une
semaine." so the string uses "a rouvert" instead of the non-standard "a
réouvert".
- Line 1679: Replace the incorrect imperfect verb and improve wording/typography
in the "message" value: change "nous adorions que vous nous suiviez" to the
conditional "nous adorerions que vous nous suiviez", replace "info
additionnelle" with the more natural "info supplémentaire", and format the unit
with a non-breaking space as "24 px" in the JSON value for the "message" key.
- Line 1669: Replace the anglicism in the fr-FR translation string: update the
"description" value that currently contains "police Geist par Vercel" to use the
correct French construction "police Geist de Vercel" (i.e., change "par" to "de"
in that string).
🪄 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: 32cc870b-6b66-44ae-b92c-13e21d241029

📥 Commits

Reviewing files that changed from the base of the PR and between 3d7b0a2 and abe9763.

📒 Files selected for processing (1)
  • i18n/locales/fr-FR.json

Comment thread i18n/locales/fr-FR.json Outdated
Comment thread i18n/locales/fr-FR.json Outdated
Comment thread i18n/locales/fr-FR.json Outdated
Comment thread i18n/locales/fr-FR.json Outdated
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: 1

🤖 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/fr-FR.json`:
- Around line 572-573: The French localization keys esm_added and esm_removed
are asymmetrically worded; update the value for "esm_added" to match the "de X à
Y" pattern used by "esm_removed" (and license_change) so both read consistently
(for example change "Le type de module a changé à ESM" to "Le type de module a
changé de CJS à ESM" or an equivalent correct source→target phrasing).
🪄 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: d3e45b5a-8a0f-43c0-989e-179706cf9d4f

📥 Commits

Reviewing files that changed from the base of the PR and between abe9763 and c6ea8e5.

📒 Files selected for processing (1)
  • i18n/locales/fr-FR.json

Comment thread i18n/locales/fr-FR.json Outdated
@WarningImHack3r
Copy link
Copy Markdown
Contributor Author

@coderabbitai resume

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai bot commented Apr 17, 2026

✅ Actions performed

Reviews resumed.

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.

Than you for your PR.
Please apply the suggested changes, that read better in French.

Comment thread i18n/locales/fr-FR.json Outdated
Comment thread i18n/locales/fr-FR.json Outdated
Comment thread i18n/locales/fr-FR.json Outdated
Comment thread i18n/locales/fr-FR.json Outdated
Comment thread i18n/locales/fr-FR.json Outdated
Comment thread i18n/locales/fr-FR.json Outdated
Comment thread i18n/locales/fr-FR.json Outdated
Comment thread i18n/locales/fr-FR.json Outdated
Comment thread i18n/locales/fr-FR.json Outdated
Comment thread i18n/locales/fr-FR.json Outdated
Co-authored-by: graphieros
@graphieros
Copy link
Copy Markdown
Contributor

@WarningImHack3r all good for you ?

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.

Merci 🌿

@graphieros graphieros added this pull request to the merge queue Apr 18, 2026
Merged via the queue into npmx-dev:main with commit ca5c9b4 Apr 18, 2026
21 checks passed
@WarningImHack3r WarningImHack3r deleted the baguette-3 branch April 18, 2026 11:26
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