Skip to content

chore(i18n): add empty translation keys generated by i18n:check - #3349

Merged
serhalp merged 1 commit into
npmx-dev:mainfrom
cylewaitforit:fix-i18n-empty-keys
Oct 23, 2026
Merged

serhalp merged 1 commit into
npmx-dev:mainfrom
cylewaitforit:fix-i18n-empty-keys

Conversation

@cylewaitforit

@cylewaitforit cylewaitforit commented Oct 22, 2026 •

Copy link
Copy Markdown
Contributor

🔗 Linked issue

🧭 Context

This commits the empty translation keys that were generated by the autofix pipeline.

📚 Description

Since these changes not actually related to the knip update and only created because the translation files were touched, this change was moved to it's own PR.

Why not stack the PRs?

PRs from forks are not currently supported by GH stacks. That would require the fix-i18n-empty-keys branch to be part of the parent repo instead of the fork. github/gh-stack#46

@agentscanapp

agentscanapp Bot commented Oct 22, 2026

Copy link
Copy Markdown

Thanks for opening this pull request! 🎉

We really appreciate you taking the time to contribute, @cylewaitforit.

A maintainer will take a look as soon as they can. In the meantime, please make sure that:

  • the description explains what changed and why
  • any related issues are linked
  • existing tests still pass

If anything needs adjusting we'll leave comments here. Thanks again!

@vercel

vercel Bot commented Oct 22, 2026 •

Copy link
Copy Markdown
Contributor

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

Project Deployment Actions Updated
npmx.dev Ready Ready Preview Oct 22, 2026 1:10pm UTC
2 Skipped Deployments
Project Deployment Actions Updated
docs.npmx.dev Ignored Ignored Preview Oct 22, 2026 1:10pm UTC
npmx-lunaria Ignored Ignored Oct 22, 2026 1:10pm UTC

Request Review

@coderabbitai

coderabbitai Bot commented Oct 22, 2026 •

Copy link
Copy Markdown
Contributor

Review in Change Stack →

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration
📥 Commits

Reviewing files that changed from the base of the PR and between 66a92c4 and 18be6ff.

📒 Files selected for processing: 8
  • i18n/locales/ar.json
  • i18n/locales/az-AZ.json
  • i18n/locales/bg-BG.json
  • i18n/locales/hu-HU.json
  • i18n/locales/kn-IN.json
  • i18n/locales/pl-PL.json
  • i18n/locales/ta-IN.json
  • i18n/locales/te-IN.json

Included review availability: This review used your included allowance. Your plan provides up to 8 included reviews per hour; 7 remain after this review.

📝 Summary

Summary by CodeRabbit

  • Localization
    • Added translation placeholders across Arabic, Azerbaijani, Bulgarian, Hungarian, Kannada, Polish, Tamil and Telugu locales for command menus, package and dependency details, charts, comparisons, sponsors, branding and other interface areas.
    • These entries provide coverage for additional interface text; the new placeholders do not include translated wording.

Eight locale files add empty translation placeholders for shared UI, charts, sponsor pages and site-wide sections. Several files also remove the nested translation_status.table object.

Changes: Locale translation namespaces

Layer / File(s) Summary
Shared UI and package namespaces
i18n/locales/{ar.json,az-AZ.json,bg-BG.json,hu-HU.json,kn-IN.json,pl-PL.json,ta-IN.json,te-IN.json}
Adds empty placeholders for command palettes, tools, dependency statistics, noodles, themes and package details.
Charts, trends and comparison namespaces
i18n/locales/{ar.json,az-AZ.json,bg-BG.json,hu-HU.json,kn-IN.json,pl-PL.json,ta-IN.json,te-IN.json}
Adds empty placeholders for timelines, chart accessibility and controls, downloads heatmaps, leaderboard likes and comparison charts or facets.
Sponsor-page namespaces
i18n/locales/{ar.json,az-AZ.json,bg-BG.json,hu-HU.json,kn-IN.json,pl-PL.json,ta-IN.json,te-IN.json}
Adds empty sponsor-page content, card and tier placeholders.
Status and site-wide namespaces
i18n/locales/{ar.json,bg-BG.json,kn-IN.json,pl-PL.json,ta-IN.json,te-IN.json}
Adds empty vacation, action-bar, logo-menu, brand and changelog placeholders. Some files replace translation_status.table with an empty translation_status object.

Suggested reviewers: trueberryless

Priority: ⬇️ Low

Merge Risk: ⚪ Minimal · up to 18be6

The locale placeholder updates are mergeable with no identified user-facing risk.

Pre-merge checks: Passed 4
Check name Status Explanation
Description check Passed The description explains that the pull request commits empty translation keys from the autofix pipeline and gives relevant context for separating it from the knip update.
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.
Title check Passed The title clearly and concisely describes the main change: adding empty i18n translation keys generated by i18n:check.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create a new PR
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

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.

@github-actions

Copy link
Copy Markdown

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 tracking.ignoredKeywords 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/ar.json Localization changed, will be marked as complete.
i18n/locales/az-AZ.json Localization changed, will be marked as complete.
i18n/locales/bg-BG.json Localization changed, will be marked as complete.
i18n/locales/hu-HU.json Localization changed, will be marked as complete.
i18n/locales/kn-IN.json Localization changed, will be marked as complete.
i18n/locales/pl-PL.json Localization changed, will be marked as complete.
i18n/locales/ta-IN.json Localization changed, will be marked as complete.
i18n/locales/te-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.

@codecov

codecov Bot commented Oct 22, 2026 •

Copy link
Copy Markdown

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!

@serhalp serhalp changed the title fix(i18n): add empty translation keys generated by i18n:check chore(i18n): add empty translation keys generated by i18n:check Oct 23, 2026
@serhalp
serhalp added this pull request to the merge queue Oct 23, 2026
Merged via the queue into npmx-dev:main with commit 7ae4670 Oct 23, 2026
27 checks passed
@github-actions github-actions Bot mentioned this pull request Oct 23, 2026
@cylewaitforit
cylewaitforit deleted the fix-i18n-empty-keys branch October 23, 2026 00:44

This branch was successfully deployed

1 active deployment
Preview – npmx.dev — 18be6ff4 Deployed Oct 22, 2026 by vercel[bot]
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