i18n(ja): restore lost code fences (batch 1, 135 files) - #23505
Conversation
Part of a corpus-wide sweep for a structural MT regression: many config/log/command/output examples in ja translations lost their triple-backtick code fences entirely, degrading to plain (sometimes still-indented) text with no syntax highlighting or copy-button UI. Restored fences to match the English source exactly, including re-indenting fence markers and body content that had accumulated stray leading whitespace from the old indented-code-block workaround. Every file's fence count and per-line indentation now matches the English source exactly; content itself is untouched. This is the first of several batches; ~216 more candidate files remain in the queue (see memory: ja-lost-code-fences-sweep). Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
|
Skipping CI for Draft Pull Request. |
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Plus Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
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 |
An independent review of the batch-1 fence restoration found 2 defect classes the automated verification missed: - check-before-deployment.md: 84 lines across ~26 command-output blocks retained a stale 4-space indent from the old workaround, not matching English's flush indentation. This file's fence tag itself had a pre-existing indentation quirk that broke the original block-pairing script's parser, causing the whole file to be silently skipped by the automated indentation-fix pass. - ai/guides/connect.md and develop/dev-guide-connection-parameters.md: 2 blockquote-nested fenced blocks (`> ``` ... > ``` `) had stale extra indentation immediately after the `> ` marker, which the original indentation checker's plain leading-whitespace comparison didn't detect (the line doesn't start with whitespace, it starts with `>`). Verified via a corpus-wide blockquote-aware line-by-line indentation check across all 135 files in this PR - no further issues found. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Continues the corpus-wide fence-restoration sweep (see PR pingcap#23505, batch 1). Same method: fence markers restored to match English exactly, including per-line indentation correction. Verified via the fence-count + positional (blockquote-aware) indentation check across all 37 files -- no remaining mismatches. 4 files from this batch's original scope (dm/dm-manage-schema.md, dm/handle-failed-ddl-statements.md, dm/manually-handling-sharding-ddl-locks.md, privilege-management.md) were still incomplete when this batch was interrupted and are reverted to their original state, to be redone cleanly in a future batch rather than committing partial/inconsistent edits. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
844d0df to
4694046
Compare
[LGTM Timeline notifier]Timeline:
|
What is changed, added or deleted? (Required)
First batch of a corpus-wide sweep fixing a structural MT regression: many config/log/command/output examples in Japanese translations lost their triple-backtick code fences entirely, degrading to plain (sometimes still-indented) text with no syntax highlighting or copy-button UI. The example content itself was untouched — only the fence markers were missing.
This PR (135 files) restores the fences to match the English source exactly:
or taggedyaml/sql/shell/```json) added back at the correct position and indentation.Verified mechanically for every file in this PR: (1) the count of
occurrences matches the English source exactly, and (2) for every matched code block, per-line leading-whitespace matches the English source exactly, including code fences nested inside blockquotes (`>... > ```), which need their own check since a naive leading-whitespace comparison does not catch indentation drift after a> ` marker.An earlier round of independent review found and fixed two defect classes the initial automated verification missed: (1) one file where a pre-existing fence-indentation quirk broke the verification script's block-parser and caused the whole file to be silently skipped, leaving ~26 blocks with stale indentation; (2) two files with the blockquote-nested-fence blind spot described above. Both are now fixed and covered by the verification method for all files in this PR.
This is the first of several planned batches — a larger candidate list (~216 more files) remains and will follow in subsequent PRs (see #23506 for batch 2). See the tracking note for full scope and method.
Which TiDB version(s) do your changes apply to? (Required)
Tips for choosing the affected version(s):
By default, CHOOSE MASTER ONLY so your changes will be applied to the next TiDB major or minor releases. If your PR involves a product feature behavior change or a compatibility change, CHOOSE THE AFFECTED RELEASE BRANCH(ES) AND MASTER.
For details, see tips for choosing the affected versions.
What is the related PR or file link(s)?
AI agent involvement
Do your changes match any of the following descriptions?