i18n(ja): fix mdsvgicon tag scrambled as wrapper across 4 files - #23654
Conversation
Same MT-scramble defect as pingcap#23653: the self-closing <MDSvgIcon /> component was scrambled into a wrapper tag holding unrelated trailing prose as fake child content, instead of matching its self-closing usage in the English source and elsewhere in the corpus. Fixes 7 occurrences in data-service-manage-data-app.md, oauth2.md, tidb-cloud-password-authentication.md (x3), and use-chat2query-api.md.
|
[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 (4)
Included review availability: Your plan provides up to 4 included reviews per hour; 1 remains after this review. 📝 WalkthroughWalkthroughThe documentation updates replace malformed lowercase icon markup with self-closing ChangesDocumentation icon corrections
Estimated code review effort: 1 (Trivial) | ~5 minutes Merge Risk: ⚪ Minimal · up to This change corrects malformed documentation component tags in four Japanese documentation files without introducing an actionable merge-blocking risk; it is merge-ready after normal checks and review. Suggested reviewers: 🚥 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. (4 skipped: 4 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:
|
First-time contributors' checklist
What is changed, added or deleted? (Required)
Same MT-scramble defect as #23653, found via a corpus-wide sweep for the remaining occurrences: the self-closing
<MDSvgIcon name="..." />component was scrambled into a wrapper tag (<mdsvgicon name="...">...unrelated trailing prose...</mdsvgicon>), instead of matching its self-closing usage in the English source and elsewhere in the corpus.Fixes 7 occurrences:
tidb-cloud/data-service-manage-data-app.md(1)tidb-cloud/oauth2.md(1)tidb-cloud/tidb-cloud-password-authentication.md(3, identical pattern)tidb-cloud/use-chat2query-api.md(1)Verified each fix against the English source (
upstream/release-8.5) and the<MDSvgIcon />self-closing convention used in sibling files.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