Skip to content

docs: fix i18n contribution guide link - #59173

Merged
QDyanbing merged 2 commits into
ant-design:masterfrom
dvd233:docs/fix-i18n-guide-link
Aug 31, 2026
Merged

QDyanbing merged 2 commits into
ant-design:masterfrom
dvd233:docs/fix-i18n-guide-link

Conversation

@dvd233

@dvd233 dvd233 commented Aug 31, 2026

Copy link
Copy Markdown
Contributor

🤔 This is a ...

  • 🆕 New feature
  • 🐞 Bug fix
  • 📝 Site / documentation improvement
  • 📽️ Demo improvement
  • 💄 Component style improvement
  • 🤖 TypeScript definition improvement
  • 📦 Bundle size optimization
  • ⚡️ Performance optimization
  • ⭐️ Feature enhancement
  • 🌐 Internationalization
  • 🛠 Refactoring
  • 🎨 Code style optimization
  • ✅ Test Case
  • 🔀 Branch merge
  • ⏩ Workflow
  • ⌨️ Accessibility improvement
  • ❓ Other (about what?)

🔗 Related Issues

N/A — this fixes an incorrect navigation target in the current contribution guide.

💡 Background and Solution

The Chinese i18n contribution guide tells contributors to update both locale lists, but both links in step 8 point to docs/react/i18n.zh-CN.md. Contributors following the Chinese guide can therefore miss the English locale list.

Update the first label and target to docs/react/i18n.en-US.md, matching the English guide and the established locale contribution workflow.

Validation:

  • Ran the repository-pinned Remark lint plugins against docs/react/i18n.zh-CN.md with zero messages.
  • Confirmed both destination URLs return HTTP 200.
  • Ran git diff --check.

📝 Change Log

Language Changelog
🇺🇸 English Fix the Chinese i18n contribution guide to link both the English and Chinese locale lists.
🇨🇳 Chinese 修复中文国际化贡献指南中的重复链接,使其分别指向中英文语言列表。

@bolt-new-by-stackblitz

Copy link
Copy Markdown

Review PR in StackBlitz Codeflow Run & review this pull request in StackBlitz Codeflow.

@github-actions

github-actions Bot commented Aug 31, 2026 •

Copy link
Copy Markdown
Contributor

Preview is ready

@github-actions

github-actions Bot commented Aug 31, 2026 •

Copy link
Copy Markdown
Contributor

👁 Visual Regression Report for PR #59173 Passed ✅

🎯 Target branch: master (c99d71d)
📖 View Full Report ↗︎
📖 Alternative Report ↗︎

🎊 Congrats! No visual-regression diff found.

📖 中文报告

👁 PR #59173 视觉回归报告 通过 ✅

🎯 目标分支: master (c99d71d)

🎊 恭喜!未发现视觉回归差异。

@coderabbitai

coderabbitai Bot commented Aug 31, 2026 •

Copy link
Copy Markdown
Contributor

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro Plus

Run ID: 109539eb-249b-4caf-805a-11944d773041

📥 Commits

Reviewing files that changed from the base of the PR and between c99d71d and 735657c.

📒 Files selected for processing (1)
  • docs/react/i18n.zh-CN.md

Included review availability: Your plan provides up to 10 included reviews per hour; 9 remain after this review.


📝 Walkthrough

Summary by CodeRabbit

  • 文档
    • 修正“增加语言包”章节中的文档链接,分别指向英文和中文国际化文档。

Walkthrough

贡献语言包指南的第 8 步已将第一个链接改为英文 i18n 文档。第二个链接继续指向中文 i18n 文档。

Changes

i18n 文档链接修正

Layer / File(s) Summary
修正贡献指南链接
docs/react/i18n.zh-CN.md
第 8 步中的第一个链接从中文文档改为英文文档。

Estimated code review effort: 1 (极低) | ~2 分钟

Merge Risk: ⚪ Minimal · up to 73565

This localized documentation change corrects the English locale-list link in the Chinese contribution guide and introduces no actionable merge-blocking risk after normal checks and review.

Poem

小兔检查链接忙,
英文中文各一方。
重复路径轻轻改,
文档导航更清爽。
胡萝卜旁盖好章。

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed 标题准确概括了主要变更:修复 i18n 贡献指南中的链接。标题简洁且与变更内容直接相关。
Description check ✅ Passed 描述明确说明了中文 i18n 贡献指南中的重复链接问题、修复方案和验证结果。内容与变更集相关。
Docstring Coverage ✅ Passed 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…
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
Full details: Docstring Coverage

Explanation

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. (1 skipped: 1 unsupported.)

✨ 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.

@pkg-pr-new

pkg-pr-new Bot commented Aug 31, 2026 •

Copy link
Copy Markdown
More templates

npm i https://pkg.pr.new/antd@59173

commit: d78cc84

@codecov

codecov Bot commented Aug 31, 2026 •

Copy link
Copy Markdown

Bundle Report

Bundle size has no change ✅

@codecov

codecov Bot commented Aug 31, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 100.00%. Comparing base (c99d71d) to head (d78cc84).

Additional details and impacted files
@@            Coverage Diff            @@
##            master    #59173   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files          834       834           
  Lines        15994     15994           
  Branches      4204      4204           
=========================================
  Hits         15994     15994           

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@QDyanbing
QDyanbing merged commit c5dbf3f into ant-design:master Aug 31, 2026
41 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants