Skip to content

i18n(ja): align heading anchors with the English source - #23325

Open
yahonda wants to merge 1 commit into
pingcap:i18n-ja-release-8.5from
yahonda:i18n-ja-fix-heading-anchors
Open

i18n(ja): align heading anchors with the English source#23325
yahonda wants to merge 1 commit into
pingcap:i18n-ja-release-8.5from
yahonda:i18n-ja-fix-heading-anchors

Conversation

@yahonda

@yahonda yahonda commented Jul 21, 2026

Copy link
Copy Markdown
Member

What is changed, added or deleted? (Required)

Align the Japanese heading {#anchor} slugs with the English source across 123 files, so that intra-document anchor links resolve correctly.

The machine-translation pipeline had regenerated many anchors from the Japanese heading text (or mangled them), leaving them out of sync with the English GitHub-generated slugs that the intra-doc links rely on. This PR restores each heading anchor to the English slug, following GitHub's slug rules:

  • Underscores are kept (e.g. `EMBED_TEXT()`{#embed_text}, not {#embed-text}).
  • + and dots collapse per GitHub (e.g. "Auto-embedding + Vector Index" → {#auto-embedding--vector-index}; "before TiDB v6.6.0" → {#...-v660}).
  • Duplicate headings keep their -1 / -2 disambiguation suffixes.
  • Angle-bracket placeholders inside code spans are preserved (e.g. `region <region_id> [--jq="<query string>"]`{#region-region_id---jqquery-string}).

Verification:

  • All 925 changed anchors were checked against the English GitHub slugs — all match.
  • Broken intra-document anchor links across these files dropped from 188 (before) to 0 (after).

Also included (same heading line, so kept in this PR to avoid a conflicting edit):

  • br/backup-and-restore-overview.md: the section heading "TiDB v6.6.0以前…" → "TiDB v6.6.0より前…", matching the English "before TiDB v6.6.0" (exclusive of v6.6.0) and the section body, which already used より前.

Only heading anchors (and the one link that pointed to a renamed anchor) are changed; no prose is retranslated.

Which TiDB version(s) do your changes apply to? (Required)

  • master (the latest development version)
  • v8.5 (TiDB 8.5 versions)
  • v8.4 (TiDB 8.4 versions)
  • v8.3 (TiDB 8.3 versions)
  • v8.2 (TiDB 8.2 versions)
  • v8.1 (TiDB 8.1 versions)
  • v7.5 (TiDB 7.5 versions)
  • v7.1 (TiDB 7.1 versions)
  • v6.5 (TiDB 6.5 versions)

What is the related PR or file link(s)?

  • This PR is translated from: N/A (Japanese-only anchor/link fix; no English source change)
  • Other reference link(s): part of the ja i18n-ja-release-8.5 machine-translation cleanup series

AI agent involvement

  • The changes in this PR were primarily made by an AI agent on behalf of the PR author.

Do your changes match any of the following descriptions?

  • Delete files
  • Change aliases
  • Need modification after applied to another branch
  • Might cause conflicts after applied to another branch

@ti-chi-bot

ti-chi-bot Bot commented Jul 21, 2026

Copy link
Copy Markdown

Skipping CI for Draft Pull Request.
If you want CI signal for your change, please convert it to an actual PR.
You can still manually trigger a test run with /test all

@ti-chi-bot ti-chi-bot Bot added the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Jul 21, 2026
@coderabbitai

coderabbitai Bot commented Jul 21, 2026

Copy link
Copy Markdown

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: 21509350-99ca-4aef-81b3-239e2e1227ed

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

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.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@ti-chi-bot

ti-chi-bot Bot commented Jul 21, 2026

Copy link
Copy Markdown

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by:
Once this PR has been reviewed and has the lgtm label, please assign kissmydb for approval. For more information see the Code Review Process.
Please ensure that each of them provides their approval before proceeding.

The full list of commands accepted by this bot can be found here.

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@ti-chi-bot ti-chi-bot Bot added area/develop This PR relates to the area of TiDB App development. missing-translation-status This PR does not have translation status info. size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files. labels Jul 21, 2026
@yahonda
yahonda force-pushed the i18n-ja-fix-heading-anchors branch 2 times, most recently from 135d6dd to abc4167 Compare July 21, 2026 09:07
@qiancai qiancai added translation/no-need No need to translate this PR. and removed missing-translation-status This PR does not have translation status info. labels Jul 21, 2026
@yahonda
yahonda force-pushed the i18n-ja-fix-heading-anchors branch 2 times, most recently from 303ac0d to 14f4cf3 Compare July 22, 2026 01:41
@yahonda
yahonda force-pushed the i18n-ja-fix-heading-anchors branch from 14f4cf3 to 42b1607 Compare July 24, 2026 01:03
Rebased onto the current i18n-ja-release-8.5 tip. Aligns Japanese heading
{#anchor} slugs with the English GitHub-generated slugs (underscores kept,
"+"/dots handled, duplicate suffixes), fixing intra-doc anchor links across
123 files (188 broken intra-doc links -> 0). Also corrects the "before TiDB
v6.6.0" heading (以前 -> より前) to match the English and the section body.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
@yahonda
yahonda force-pushed the i18n-ja-fix-heading-anchors branch from 3ab452f to 9dcd279 Compare July 30, 2026 05:43
@yahonda
yahonda marked this pull request as ready for review July 30, 2026 05:43
@ti-chi-bot ti-chi-bot Bot removed the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Jul 30, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/develop This PR relates to the area of TiDB App development. size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files. translation/no-need No need to translate this PR.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants