Complete Simplified Chinese localization - #48
Conversation
|
Caution The consumer version of Gemini Code Assist on GitHub has been sunset. All code review activity has officially ceased. |
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (1)
🚧 Files skipped from review as they are similar to previous changes (1)
📝 WalkthroughWalkthroughUpdated Simplified Chinese translations for navigation, app details, security checks, updates, authentication, donations, Telegram, favorites, settings, and category labels. ChangesSimplified Chinese localization
Estimated code review effort: 2 (Simple) | ~10 minutes Possibly related PRs
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
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. Comment |
|
Caution The consumer version of Gemini Code Assist on GitHub has been sunset. All code review activity has officially ceased. |
There was a problem hiding this comment.
Actionable comments posted: 4
🤖 Prompt for all review comments with AI agents
Verify 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 `@app/src/main/res/values-zh-rCN/strings.xml`:
- Line 175: Update the strings at the referenced entries to use the canonical
app name “Repo Store” instead of “RepoStore,” preserving the existing Simplified
Chinese wording and placeholders.
- Around line 286-288: Update the Chinese translations for select_variant,
compatible, and incompatible: use “选择变体” for select_variant, “兼容” for
compatible, and “不兼容” for incompatible, matching the APK variant terminology
used by no_apk_variants_available.
- Around line 279-281: Replace the literal term in the multi-part download
strings with the standard `分段下载` wording, updating `multi_part_download`,
`enable_multi_part`, and `download_settings_info` consistently while preserving
the rest of each translation.
- Line 90: Update the security_unknown string resource in the zh-rCN locale to
use the unknown-state translation, such as 未知, rather than the GitHub release
label used by security_source.
🪄 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: defaults
Review profile: CHILL
Plan: Pro Plus
Run ID: a7570c9e-4bde-4079-bcce-706f9086d0e7
📒 Files selected for processing (1)
app/src/main/res/values-zh-rCN/strings.xml
Summary
Scope
This PR is limited to the existing Simplified Chinese Android localization (
app/src/main/res/values-zh-rCN/strings.xml).It does not change application behavior, English source strings, other locales, dependencies, or generated output.
Validation
sh gradlew --no-daemon test— 32 tests passedsh gradlew --no-daemon assembleDebug— debug APK built successfullygit diff --check— passedsh gradlew --no-daemon lint— the repository-wide task still exits with 130 pre-existingMissingTranslationerrors from other sparse locales, reproduced on cleanmain; Simplified Chinese-related errors decreased from 46 to 0, with no newzh-rCNwarningsLocalization QA
460340passed on Pixel_7 emulator (API 37) — Home, Search, App details, Settings, App Updates, Download settings, Language dialog, Donate rendered in Simplified Chinese with no English fallback leakageNotes
The existing Simplified Chinese localization was retained wherever it was already accurate. Broader style preferences and low-confidence terminology changes were deliberately excluded.
Summary by CodeRabbit