i18n(ja): fix dropped/duplicated particles in backported release-note bullets - #23708
i18n(ja): fix dropped/duplicated particles in backported release-note bullets#23708yahonda wants to merge 4 commits into
Conversation
|
Skipping CI for Draft Pull Request. |
ti-chi-bot
Bot
added
the
do-not-merge/work-in-progress
|
Warning Review limit reachedNext included review available in 53 minutes. View limit detailsLimit details: You’ve used all 4 included reviews currently available. You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository. Review configuration: ⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: ASSERTIVE Plan: Team Run ID: 📒 Files selected for processing (7)
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 (7)
Included review availability: Your plan provides up to 4 included reviews per hour; 1 remains after this review. 📝 WalkthroughWalkthroughThe release notes contain Japanese wording corrections in ten entries across seven version documents. The changes affect grammar and phrasing only. No functional, structural, or public entity changes were made. ChangesRelease note corrections
Estimated code review effort: 1 (Trivial) | ~2 minutes Merge Risk: ⚪ Minimal · up to This change only corrects wording in Japanese release notes and does not alter product behavior; no actionable merge-blocking risk remains beyond normal checks and review. 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
Full details: Description checkExplanation The description explains the changes, affected files, verification method, target documentation branch, and AI involvement. The selected version is stated as the repository-specific 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. (7 skipped: 7 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 |
ti-chi-bot
Bot
added
the
missing-translation-status
|
[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 |
ti-chi-bot
Bot
added
the
size/S
ti-chi-bot
Bot
removed
the
do-not-merge/work-in-progress
cd024ad to
9deafab
Compare
qiancai
added
translation/no-need
[LGTM Timeline notifier]Timeline:
|
The bullet was missing the verb する connecting "log wo shutsuryoku" (output the log) to "mondai" (problem), reading as a broken noun compound instead of a relative clause. Same bug backported identically to 4 release files. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
… release-7.4.0.md EN: "when IndexLookup involves many table retrieval tasks". JA was missing the ni-particle marking IndexLookup as the container/subject of the clause. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
The bullet had a redundant "de" stacked right after "to" (jikkousuru to de), an MT artifact; the sibling occurrence of the same bug in release-8.2.0.md was already phrased correctly. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
…convention EN: "Fix the overflow issue of the Longlong type in predicates". The possessive no-particle (predicate's Longlong type) subtly misstates this as the predicate's own type rather than a type used within predicates. Changed to niokeru, matching the sibling occurrence in release-8.2.0.md and the established convention used elsewhere in the corpus for "type error in X" bullets (e.g. release-4.0.11.md's "function niokeru duration type inference" bullet). Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
9deafab to
35f2126
Compare
2 participants
What is changed, added or deleted? (Required)
Found while reading through release notes: several bullets (mostly backported bug-fix descriptions appearing identically across multiple release branches) had dropped or duplicated particles, or a particle choice that subtly misstated the relationship EN describes.
Fixed:
IndexLookupin release-7.4.0.md's cost-model bulletREMOVE PARTITIONINGbullet (sibling release-8.2.0.md already had it correct)Longlongoverflow-in-predicates bullet across 4 files (release-6.5.10.md, release-7.1.6.md, release-7.5.2.md, release-8.1.1.md), matching the already-correct sibling in release-8.2.0.md and the established corpus convention for "type error in X" bulletsEach fix was verified against the corresponding EN source in
release-8.5, using the issue/PR number as an anchor and cross-checking sibling occurrences of the same backported bug across release branches.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