i18n(ja): fix stray text fragments and scrambled link lists - #23640
Conversation
Found via a corpus-wide scan for a recurring MT-pipeline defect: a short text fragment duplicated or misplaced immediately before a markdown link, with no counterpart in the English source. - control-execution-plan.md: removes a stray "SQL " duplicated before the "SQL Plan Management" link, and restores a missing opening backtick around `tidb_opt_fix_control` in the frontmatter summary. - 6 sample-application "Next steps" guides (nodejs-typeorm, nodejs-prisma, nodejs-mysql2, nodejs-mysqljs, ruby-rails, ruby-mysql2) shared an identical scrambled chapter-list sentence: missing commas between several links, a stray "SQL" fragment before the "Transactions" link, and (in 2 of the 6 files) the "Query data" link displaced to the wrong position or splitting a verb in half. Reconstructed all 6 to a single corrected sentence matching the English source's list order and structure.
|
[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 |
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: ASSERTIVE Plan: Pro Plus Run ID: 📒 Files selected for processing (7)
Included review availability: Your plan provides up to 4 included reviews per hour; 3 remain after this review. 📝 WalkthroughWalkthroughDocumentation-only changes correct inline code formatting and Japanese developer-guide links across the control execution plan and sample application guides. ChangesDocumentation corrections
Estimated code review effort: 1 (Trivial) | ~5 minutes Merge Risk: ⚪ Minimal · up to This PR makes localized Japanese documentation corrections without changing product behavior, and no actionable merge-blocking risk remains beyond normal checks and review. 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
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 |
[LGTM Timeline notifier]Timeline:
|
What is changed, added or deleted? (Required)
Found via a corpus-wide scan for a recurring MT-pipeline defect: a short text fragment duplicated or misplaced immediately before a markdown link, with no counterpart in the English source (the same defect class as the stray "オプティマ" fragment fixed in #23630 and the stray "SI" fragments fixed in #23639).
control-execution-plan.md: removes a straySQLduplicated before the[SQLプラン管理]link (EN: "In SQL Plan Management", no bare "SQL" before it), and restores a missing opening backtick around`tidb_opt_fix_control`in the frontmatter summary.nodejs-typeorm,nodejs-prisma,nodejs-mysql2,nodejs-mysqljs,ruby-rails,ruby-mysql2) shared an identical scrambled chapter-list sentence: missing commas between several links, a straySQLfragment before the[トランザクション](Transactions) link, and (in 2 of the 6 files) the[クエリデータ](Query data) link displaced to the wrong position or splitting a verb in half. Reconstructed all 6 to a single corrected sentence matching the English source's list order and structure (Insert data, Update data, Delete data, Query data, Transactions, SQL performance optimization).7 files, 7 sites.
Method
Scanned the whole corpus for two shapes: (1) a JA text run immediately abutting a link where the run is a proper prefix of the link's own text, and (2) a short (2-5 char) token set off by a space immediately before a link. Most candidates from both scans were legitimate (e.g.
TiFlash [use TiFlash],PD [config item],IETF [RFC title]— a real qualifying word genuinely preceding the link) and were left untouched; only sites verified against the exact EN sentence to have no counterpart for the leading fragment were fixed.Verified
markdownlint-cli2clean on all touched files.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