i18n(ja): restore particles dropped after code spans and links#23318
i18n(ja): restore particles dropped after code spans and links#23318yahonda wants to merge 1 commit into
Conversation
|
Skipping CI for Draft Pull Request. |
|
Important Review skippedDraft detected. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing Touches🧪 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 |
ca11335 to
a73819e
Compare
2b321d6 to
f5b3208
Compare
The old MT pipeline dropped the particle between a code span (or link) and the verb that governs it. The correct particle depends on voice, which the English source settles: EN "See [X](/x.md)" -> [X](/x.md)を参照してください EN "`200 OK` is returned" -> `200 OK`が返されます (passive: が, not を) Every site was read against release-8.5 before editing. Verified mechanically: each changed line differs from the original by inserted particles only — zero deletions, so every code span, link URL and markdown structure is byte-identical. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
f5b3208 to
3a16df0
Compare
What is changed, added or deleted? (Required)
The old MT pipeline dropped the particle between a code span (or link) and the verb that governs it. The correct particle depends on grammatical voice, so every site was read against the English source rather than pattern-replaced:
433 lines across 212 files. Particle distribution:
INFORMATION_SCHEMA.CLUSTER_HARDWAREmight contain …" ->に…含まれている-n/--namespecifies" ->で); correlative ("neither<version>nor--all" ->も)28% of the sites are not を. A mechanical
をinsertion would have been wrong on those 112 lines — which is why #23272 deliberately excluded passive forms and left them for a pass like this one.How it was checked
Not fixed
About a dozen flagged sites are damaged beyond a missing particle — the pipeline transposed or duplicated code spans, so no particle is correct. Examples:
optimizer-hints.md(ヒントをalready holds the accusative, soQB_NAMEhas no valid slot),br/br-checkpoint-restore.md(thebrspan is duplicated, giving two subjects),partition-pruning.md(the operator list is torn apart). These need rewording, not particle restoration, and are left for a follow-up.Relationship to the sibling PRs
These fixes were developed on top of the other open ja PRs and then harmonized with them, so all thirteen merge cleanly in any order (verified: 78 pairwise merges, 0 conflicts). Where a line is shared or adjacent, every branch carries the same final text — which is why 13 lines here also contain a sibling PR's fix (a stray digit removed, a stranded
}}dropped).Which TiDB version(s) do your changes apply to? (Required)
What is the related PR or file link(s)?
Do your changes match any of the following descriptions?