i18n(ja): restore literal English text for user-facing error messages - #23695
i18n(ja): restore literal English text for user-facing error messages#23695yahonda wants to merge 3 commits into
Conversation
|
Skipping CI for Draft Pull Request. |
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: ASSERTIVE Plan: Team Run ID: 📒 Files selected for processing (14)
Included review availability: Your plan provides up to 4 included reviews per hour; 3 remain after this review. 📝 WalkthroughWalkthroughChangesError message wording updates
Estimated code review effort: 1 (Trivial) | ~5 minutes Merge Risk: ⚪ Minimal · up to Japanese documentation now preserves the corresponding English error strings across release notes, troubleshooting, FAQ, and configuration guidance. No behavior, configuration semantics, or documentation links are changed, so the update is ready to merge. Suggested reviewers: 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
Full details: Description checkExplanation The description explains the change, rationale, affected files, source verification, target documentation version, and AI involvement. The related-link fields are blank, but this is non-critical because no specific link is required for this documentation-only change. Full details: Docstring CoverageExplanation No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0 files. (14 skipped: 14 unsupported.) ✨ Finishing Touches 💡 1🛠️ Fix failing CI checks 💡
🧪 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 |
|
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: The full list of commands accepted by this bot can be found here. DetailsNeeds approval from an approver in each of these files:Approvers can indicate their approval by writing |
c8de5ec to
bd2c922
Compare
Several release notes and reference pages had translated the actual English error message/string TiDB or its tools emit into paraphrased Japanese, instead of keeping it literal (matching EN's own quoted error text). Fixed 21 occurrences across 14 files. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
release-5.0.0.md and release-5.1.0.md used TiDB構成 while the identical templated sentence in release-5.2.0.md already used TiDB 設定, matching the corpus-wide 構成->設定 unification decision. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
The second bold span had drifted onto "ください" in the first sentence instead of marking "使用しないでください" (matching EN's second **DO NOT**), leaving the actual DO-NOT clause unemphasized. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
bd2c922 to
d8f9ed1
Compare
|
Note GitHub couldn't provide a complete incremental comparison for this pull request, so CodeRabbit is performing a full review instead. This review may take a little longer. |
What is changed, added or deleted? (Required)
Several release notes and reference pages had the actual English error message/string that TiDB (or BR/TiDB Lightning/TiFlash) emits translated into paraphrased Japanese, instead of being kept literal as it appears in EN. Since these are real error strings a user would see verbatim, the Japanese translation should keep them in English too.
Fixed 21 occurrences across 14 files:
releases/release-2.0.1.md,release-2.1-rc.2.md,release-3.0.1.md,release-4.0.14.md,release-4.0.15.md(2),release-5.0.0.md,release-5.1.0.md(2),release-5.2.0.md(2),release-5.4.1.md(2),release-6.5.3.md,release-7.1.3.mdtidb-lightning/data-import-best-practices.md(4)tikv-configuration-file.mdtidb-cloud/serverless-faqs.mdEach fix was verified against the corresponding EN source in
release-8.5to confirm EN itself quotes the literal error text.Which TiDB version(s) do your changes apply to? (Required)
What is the related PR or file link(s)?
AI agent involvement
Do your changes match any of the following descriptions?
Summary by CodeRabbit