Skip to content

i18n(ja): convert full-width hash to half-width in issue/PR references - #23597

Closed
yahonda wants to merge 7 commits into
pingcap:i18n-ja-release-8.5from
yahonda:fix-fullwidth-hash-to-halfwidth
Closed

i18n(ja): convert full-width hash to half-width in issue/PR references#23597
yahonda wants to merge 7 commits into
pingcap:i18n-ja-release-8.5from
yahonda:fix-fullwidth-hash-to-halfwidth

Conversation

@yahonda

@yahonda yahonda commented Aug 25, 2026

Copy link
Copy Markdown
Member

What is changed, added or deleted? (Required)

The Japanese translation rendered the # in GitHub issue/PR reference link text (e.g. [#12345]) as the full-width CJK character # instead of the half-width # used throughout the English source and in the actual linked URLs.

Converts all 9024 occurrences of # to half-width # across 166 files. This includes 42 occurrences in resources/doc-templates/patch_release_note_template_zh.md's placeholder text ([#問題]), where the character isn't followed by digits but still represents the same GitHub issue-number marker.

Verified: 0 remaining full-width # occurrences, line-count-symmetric diff (8510 lines changed, no additions/removals).

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

  • i18n-ja-release-8.5 (TiDB Japanese documentation for TiDB 8.5 versions)

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

  • This PR is translated from:
  • Other reference link(s):

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

…erences

The Japanese translation rendered the # in GitHub issue/PR link text
(e.g. [#12345]) as the full-width CJK character # instead of the
half-width # used throughout the English source and in the actual
linked URLs. Converts all 9024 occurrences across 166 files to match.

Also fixes 42 occurrences of the same pattern in resources/doc-
templates/patch_release_note_template_zh.md's placeholder text
([#問題]), where the character wasn't followed by digits but still
represents the same GitHub issue-number marker.
@coderabbitai

coderabbitai Bot commented Aug 25, 2026

Copy link
Copy Markdown

Important

Review skipped

Too many files!

This PR contains 166 files, which is 16 over the limit of 150.

To get a review, reduce the PR to 150 files or fewer by splitting it into smaller PRs or changing its base branch.

Upgrade to a paid plan to raise the limit.

This review couldn't start because sufficient usage credits or metered capacity aren't available. Add credits or update usage-based reviews in the billing tab, then retry.

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: ASSERTIVE

Plan: Pro Plus

Run ID: 173b860d-9789-42de-8227-5b0f57242d22

📥 Commits

Reviewing files that changed from the base of the PR and between 603a4d9 and 552c159.

📒 Files selected for processing (166)
  • br/br-pitr-manual.md
  • br/br-snapshot-manual.md
  • faq/backup-and-restore-faq.md
  • functions-and-operators/aggregate-group-by-functions.md
  • functions-and-operators/bit-functions-and-operators.md
  • functions-and-operators/cast-functions-and-operators.md
  • functions-and-operators/json-functions.md
  • generated-columns.md
  • releases/release-2.0.10.md
  • releases/release-2.0.11.md
  • releases/release-2.0.5.md
  • releases/release-2.0.6.md
  • releases/release-2.0.7.md
  • releases/release-2.0.8.md
  • releases/release-2.0.9.md
  • releases/release-2.1-rc.1.md
  • releases/release-2.1-rc.2.md
  • releases/release-2.1-rc.3.md
  • releases/release-2.1-rc.4.md
  • releases/release-2.1-rc.5.md
  • releases/release-2.1.1.md
  • releases/release-2.1.10.md
  • releases/release-2.1.11.md
  • releases/release-2.1.12.md
  • releases/release-2.1.13.md
  • releases/release-2.1.14.md
  • releases/release-2.1.15.md
  • releases/release-2.1.16.md
  • releases/release-2.1.17.md
  • releases/release-2.1.18.md
  • releases/release-2.1.19.md
  • releases/release-2.1.2.md
  • releases/release-2.1.3.md
  • releases/release-2.1.4.md
  • releases/release-2.1.5.md
  • releases/release-2.1.6.md
  • releases/release-2.1.7.md
  • releases/release-2.1.8.md
  • releases/release-2.1.9.md
  • releases/release-3.0-beta.md
  • releases/release-3.0.0-beta.1.md
  • releases/release-3.0.0-rc.1.md
  • releases/release-3.0.0-rc.2.md
  • releases/release-3.0.0-rc.3.md
  • releases/release-3.0.1.md
  • releases/release-3.0.10.md
  • releases/release-3.0.11.md
  • releases/release-3.0.12.md
  • releases/release-3.0.13.md
  • releases/release-3.0.14.md
  • releases/release-3.0.15.md
  • releases/release-3.0.16.md
  • releases/release-3.0.17.md
  • releases/release-3.0.18.md
  • releases/release-3.0.19.md
  • releases/release-3.0.2.md
  • releases/release-3.0.20.md
  • releases/release-3.0.3.md
  • releases/release-3.0.4.md
  • releases/release-3.0.5.md
  • releases/release-3.0.6.md
  • releases/release-3.0.7.md
  • releases/release-3.0.8.md
  • releases/release-3.0.9.md
  • releases/release-3.1.0-beta.1.md
  • releases/release-3.1.0-beta.2.md
  • releases/release-3.1.0-beta.md
  • releases/release-3.1.0-ga.md
  • releases/release-3.1.0-rc.md
  • releases/release-3.1.1.md
  • releases/release-3.1.2.md
  • releases/release-4.0-ga.md
  • releases/release-4.0.0-beta.1.md
  • releases/release-4.0.0-beta.2.md
  • releases/release-4.0.0-beta.md
  • releases/release-4.0.0-rc.1.md
  • releases/release-4.0.0-rc.2.md
  • releases/release-4.0.0-rc.md
  • releases/release-4.0.1.md
  • releases/release-4.0.10.md
  • releases/release-4.0.11.md
  • releases/release-4.0.12.md
  • releases/release-4.0.13.md
  • releases/release-4.0.14.md
  • releases/release-4.0.15.md
  • releases/release-4.0.16.md
  • releases/release-4.0.2.md
  • releases/release-4.0.3.md
  • releases/release-4.0.4.md
  • releases/release-4.0.5.md
  • releases/release-4.0.6.md
  • releases/release-4.0.7.md
  • releases/release-4.0.8.md
  • releases/release-4.0.9.md
  • releases/release-5.0.0-rc.md
  • releases/release-5.0.1.md
  • releases/release-5.0.2.md
  • releases/release-5.0.3.md
  • releases/release-5.0.4.md
  • releases/release-5.0.5.md
  • releases/release-5.0.6.md
  • releases/release-5.1.1.md
  • releases/release-5.1.2.md
  • releases/release-5.1.3.md
  • releases/release-5.1.4.md
  • releases/release-5.2.1.md
  • releases/release-5.2.2.md
  • releases/release-5.2.3.md
  • releases/release-5.3.0.md
  • releases/release-5.3.1.md
  • releases/release-5.3.2.md
  • releases/release-5.3.3.md
  • releases/release-5.3.4.md
  • releases/release-5.4.1.md
  • releases/release-5.4.2.md
  • releases/release-5.4.3.md
  • releases/release-6.0.0-dmr.md
  • releases/release-6.1.0.md
  • releases/release-6.1.1.md
  • releases/release-6.1.2.md
  • releases/release-6.1.3.md
  • releases/release-6.1.4.md
  • releases/release-6.1.5.md
  • releases/release-6.1.6.md
  • releases/release-6.1.7.md
  • releases/release-6.5.0.md
  • releases/release-6.5.1.md
  • releases/release-6.5.10.md
  • releases/release-6.5.11.md
  • releases/release-6.5.12.md
  • releases/release-6.5.2.md
  • releases/release-6.5.3.md
  • releases/release-6.5.4.md
  • releases/release-6.5.5.md
  • releases/release-6.5.6.md
  • releases/release-6.5.7.md
  • releases/release-6.5.8.md
  • releases/release-6.5.9.md
  • releases/release-7.1.0.md
  • releases/release-7.1.1.md
  • releases/release-7.1.2.md
  • releases/release-7.1.3.md
  • releases/release-7.1.4.md
  • releases/release-7.1.5.md
  • releases/release-7.1.6.md
  • releases/release-7.4.0.md
  • releases/release-7.5.1.md
  • releases/release-7.5.2.md
  • releases/release-7.5.3.md
  • releases/release-7.5.4.md
  • releases/release-7.5.5.md
  • releases/release-7.5.6.md
  • releases/release-7.5.7.md
  • releases/release-8.1.0.md
  • releases/release-8.1.1.md
  • releases/release-8.1.2.md
  • resources/doc-templates/patch_release_note_template_zh.md
  • smooth-upgrade-tidb.md
  • sql-statements/sql-statement-analyze-table.md
  • support.md
  • ticdc/ticdc-avro-protocol.md
  • ticdc/ticdc-split-update-behavior.md
  • tiflash-upgrade-guide.md
  • tiflash/tiflash-configuration.md
  • tiup/tiup-command-mirror-rotate.md
  • troubleshoot-cpu-issues.md

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


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 Aug 25, 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 hfxsd 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 missing-translation-status This PR does not have translation status info. size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files. labels Aug 25, 2026
…e notes

- release-2.0.10.md, release-2.1.1.md (x2): "しないでください" reads as
  a command to the reader, but the English source ("Avoid transferring
  the leader...") describes a system behavior change, not a user
  instruction. Changed to "しないようにします" to match.
- release-2.0.5.md: missing が and を particles, and 必要となる (which
  needs が) mismatched with the を already present; fixed to the
  consistent transitive form 必要とする.
…notes

- release-2.0.9.md: 構成→設定 for "configuration" (matches the pingcap#23596 direction decision)
- release-2.1-rc.1.md: 3 missing を/の particles, plus fixed scrambled
  word order in the Scope-check bullet
…r in 10 release notes

Found by independent 1:1 EN review of this PR's changed files:
- 5 misleading imperative/cohortative renderings ("してください"/
  "しましょう") of English descriptions of tool/system behavior changes,
  not user instructions
- 5 issue-link references wedged mid-sentence, splitting a
  paired/compound term (e.g. ENUM/SET, start-ts/checkpoint-ts) instead
  of sitting at the end of the bullet
- release-3.0.20.md: typo 削除れない → 削除しない (missing さ) plus
  restructured sentence
- release-3.1.0-ga.md: spurious dangling が particle with no referent
- release-6.5.10.md/7.1.6.md/7.5.2.md: rewrote a heavily garbled
  paragraph (spurious ordinal 目/件目 counters, a fabricated "13件目",
  dangling してください with no governing verb) to match the English
  source's TiCDC UPDATE-split-behavior description (pingcap#10918)
- ticdc/ticdc-split-update-behavior.md: removed a stray つ inserted
  after every `DELETE`と`INSERT` mention (6 sites)
- release-7.1.0.md, release-7.1.3.md: removed spurious unclosed full-
  width parentheses not present in the English source
- release-7.5.4.md: fixed a garbled trailing clause ("in some cases")
  that had become a dangling 場合) with an unmatched parenthesis
- release-6.5.12.md/7.5.5.md/8.1.2.md: fixed duplicated の (負のの→負の)
- release-8.1.1.md: misleading imperative 変更してください→変更します
  (describes a change TiDB itself made, not a user instruction)
- release-6.5.3.md: a literal BR error-message string ("resolve lock
  timeout") had been translated into Japanese, misrepresenting what
  actually appears in the tool's output; restored to the literal string
- release-6.5.11.md/7.1.6.md/7.5.3.md: 構成項目→設定項目 for the same
  referent repeated two sentences later as 設定項目 (matches the pingcap#23596
  direction; these release-note files were missed by that sweep)
@yahonda

yahonda commented Aug 25, 2026

Copy link
Copy Markdown
Member Author

Splitting this PR into two smaller PRs to stay under CodeRabbit's 150-file review limit (this PR touched 166 files, 16 over the limit):

Both are byte-identical to this PR's content for their respective file subsets. Closing in favor of the two split PRs.

@yahonda yahonda closed this Aug 25, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

missing-translation-status This PR does not have translation status info. size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant