Skip to content

feat(i18n): update hi-IN and mr-IN translation for npmx tagline#2567

Merged
graphieros merged 2 commits intonpmx-dev:mainfrom
trivikr:npmx-tagline-hi-mr-in
Apr 19, 2026
Merged

feat(i18n): update hi-IN and mr-IN translation for npmx tagline#2567
graphieros merged 2 commits intonpmx-dev:mainfrom
trivikr:npmx-tagline-hi-mr-in

Conversation

@trivikr
Copy link
Copy Markdown
Contributor

@trivikr trivikr commented Apr 18, 2026

🔗 Linked issue

Refs: #1689

🧭 Context

Adds hi-IN and mr-IN translation for updated tagline

a fast, modern browser for the npm registry

📚 Description

Adds hi-IN and mr-IN translation for updated tagline

a fast, modern browser for the npm registry

@vercel
Copy link
Copy Markdown

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

Request Review

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai bot commented Apr 18, 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: 1e07be25-88c5-44b1-8eaf-ae9207f016c6

📥 Commits

Reviewing files that changed from the base of the PR and between f24b9ed and 6706f6c.

📒 Files selected for processing (2)
  • i18n/locales/hi-IN.json
  • i18n/locales/mr-IN.json
✅ Files skipped from review due to trivial changes (2)
  • i18n/locales/hi-IN.json
  • i18n/locales/mr-IN.json

📝 Walkthrough

Summary by CodeRabbit

  • Localization
    • Updated Hindi translations: refined the app tagline and homepage SEO description to emphasise a faster, modern browser experience.
    • Updated Marathi translations: refined the app tagline and homepage SEO description to emphasise a faster, modern browser experience.

Walkthrough

Updated Hindi and Marathi locale files: seo.home.description and top-level tagline values changed to describe the npm registry browser as "fast and modern" instead of "better". No other keys or structure were modified.

Changes

Cohort / File(s) Summary
i18n Locale Updates
i18n/locales/hi-IN.json, i18n/locales/mr-IN.json
Revised two translation values per file: seo.home.description and tagline updated to use wording equivalent to "fast and modern" instead of "better". No structural or placeholder changes.
🚥 Pre-merge checks | ✅ 2
✅ Passed checks (2 passed)
Check name Status Explanation
Title check ✅ Passed The pull request title accurately describes the main change: updating Hindi and Marathi translations for the npmx tagline to reflect 'a fast, modern browser for the npm registry'.
Description check ✅ Passed The pull request description is directly related to the changeset, clearly explaining the intent to add hi-IN and mr-IN translations for the updated tagline with a reference to the linked issue.

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

github-actions bot commented Apr 18, 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/hi-IN.json Localization changed, will be marked as complete.
i18n/locales/mr-IN.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.

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.

🧹 Nitpick comments (2)
i18n/locales/mr-IN.json (1)

6-6: Consider updating SEO description for consistency.

The seo.home.description still uses "एक चांगला ब्राउझर" (a good browser), whilst the tagline now uses "वेगवान आणि आधुनिक" (fast and modern). Whilst this isn't blocking (since the PR is specifically about the tagline), you might want to update this for consistency across the user experience.

🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.

In `@i18n/locales/mr-IN.json` at line 6, The seo.home.description value in
i18n/locales/mr-IN.json currently says "एक चांगला ब्राउझर"; update the string
for the key seo.home.description to reflect the new tagline by replacing "एक
चांगला ब्राउझर" with phrasing that includes "वेगवान आणि आधुनिक" (e.g., "npm
नोंदणीसाठी वेगवान आणि आधुनिक ब्राउझर. आधुनिक इंटरफेससह पॅकेजेस शोधा, ब्राउझ करा
आणि एक्सप्लोर करा.") so the description is consistent with the updated tagline.
i18n/locales/hi-IN.json (1)

6-6: Consider updating SEO description for consistency.

The seo.home.description still uses "एक बेहतर ब्राउज़र" (a better browser), whilst the tagline now uses "तेज़ और आधुनिक" (fast and modern). Whilst this isn't blocking (since the PR is specifically about the tagline), you might want to update this for consistency across the user experience.

🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.

In `@i18n/locales/hi-IN.json` at line 6, Update the seo.home.description string to
match the new tagline tone ("fast and modern") for consistency; specifically
edit the JSON key "seo.home.description" (currently "एक बेहतर ब्राउज़र…") to a
Hindi phrase that conveys "fast and modern" (e.g., include तेज़ और आधुनिक) while
keeping the rest of the sentence about browsing/searching npm registry.
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.

Nitpick comments:
In `@i18n/locales/hi-IN.json`:
- Line 6: Update the seo.home.description string to match the new tagline tone
("fast and modern") for consistency; specifically edit the JSON key
"seo.home.description" (currently "एक बेहतर ब्राउज़र…") to a Hindi phrase that
conveys "fast and modern" (e.g., include तेज़ और आधुनिक) while keeping the rest
of the sentence about browsing/searching npm registry.

In `@i18n/locales/mr-IN.json`:
- Line 6: The seo.home.description value in i18n/locales/mr-IN.json currently
says "एक चांगला ब्राउझर"; update the string for the key seo.home.description to
reflect the new tagline by replacing "एक चांगला ब्राउझर" with phrasing that
includes "वेगवान आणि आधुनिक" (e.g., "npm नोंदणीसाठी वेगवान आणि आधुनिक ब्राउझर.
आधुनिक इंटरफेससह पॅकेजेस शोधा, ब्राउझ करा आणि एक्सप्लोर करा.") so the
description is consistent with the updated tagline.

ℹ️ Review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: bbe46725-b469-4ca5-b84b-98374c233d69

📥 Commits

Reviewing files that changed from the base of the PR and between e365dd4 and f24b9ed.

📒 Files selected for processing (2)
  • i18n/locales/hi-IN.json
  • i18n/locales/mr-IN.json

@codecov
Copy link
Copy Markdown

codecov bot commented Apr 18, 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!

@graphieros graphieros added this pull request to the merge queue Apr 19, 2026
Merged via the queue into npmx-dev:main with commit 400ddf8 Apr 19, 2026
21 checks passed
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