| FazBrowse GitHub Viewer | Trending | | Home |
| Tools: [Download Repo ZIP] [Original HTTPS Page] |
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
Sorry, something went wrong.
📝 Walkthrough
WalkthroughThis pull request expands Japanese localisation across multiple UI features in a single file. It adds new navigation shortcuts and command palette entries; introduces a complete new noodles section; extends package stats, replacement, and card metadata translations; revises trend chart terminology and copy; and enhances compare UI with GitHub forks facet and diff-handling strings. ChangesJapanese Localisation Updates
Possibly related PRs
Suggested reviewers
Important Pre-merge checks failedPlease resolve all errors before merging. Addressing warnings is optional. ❌ Failed checks (1 inconclusive)
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. ❤️ ShareComment @coderabbitai help to get the list of available commands and usage tips. |
Sorry, something went wrong.
Lunaria Status Overview🌕 This pull request will trigger status changes. Learn moreBy 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
|
Sorry, something went wrong.
Codecov Report✅ All modified and coverable lines are covered by tests. 📢 Thoughts on this report? Let us know! |
Sorry, something went wrong.
|
Do you think you could also add the other 36 missing translations keys?
(package.card.version was already added in this PR) |
Sorry, something went wrong.
|
Oh, I tried https://npmx.dev/translation-status for the first time and I thought there is only one missing key, but this page actually reflects the only stable version? 🤔
Anyway, I'm going to translate them all soon today. |
Sorry, something went wrong.
Yeah, unfortunately the current state from main doesn't load under https://main.npmx.dev/translation-status, so you have to work with our i18n commands to be sure... I ran node scripts/compare-translations.ts ja-JP to get the output 👍 |
Sorry, something went wrong.
There was a problem hiding this comment.
^^
Sorry, something went wrong.
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agentsVerify each finding against current code. Fix only still-valid issues, skip the rest with a brief reason, keep changes minimal, and validate. Inline comments: In `@i18n/locales/ja-JP.json`: - Around line 158-159: The "changelog" label in the locale file is left in English while adjacent command labels like "stats" are translated to Japanese for consistency. Translate the value of the "changelog" key from "Changelog" to its Japanese equivalent (suggested: "変更履歴") to maintain consistent localization throughout the menu.
Fix all unresolved CodeRabbit comments on this PR:
Configuration used: Path: .coderabbit.yaml
Review profile: CHILL
Plan: Pro
Run ID: 138de581-cac4-479e-8c9c-ce686ce5853a
📥 CommitsReviewing files that changed from the base of the PR and between 7838bec and 3df327f.
📒 Files selected for processing (1)
Sorry, something went wrong.
|
@trueberryless Sure! I'll kick off a full review of the PR right away. ✅ Action performedFull review finished. |
Sorry, something went wrong.
There was a problem hiding this comment.
Thank you very much for taking the time to also add the additional translation strings. Much appreciated 💚
As a non-native speaker, I confirm that all translation strings are hereby up-to-date in Japanese 🙌
Sorry, something went wrong.
|
Thanks! I saw the attempt to fix https://main.npmx.dev/translation-status at #2860. Once we found the solution, I think we can show the same latest translation status on both URL to avoid confusion. |
Sorry, something went wrong.
| Back | FazBrowse Home | New Git URL |
🔗 Linked issue
N/A
🧭 Context
There was a small error in the download chart alt text transtation where there are two "。" (period equivalent in Japanese) in the middle of the alt text.
📚 Description