Skip to content

docs: fix Chinese mining guide relative links#6714

Merged
Scottcjn merged 1 commit into
Scottcjn:mainfrom
yyswhsccc:bounty-radar/issue-self-mining-guide-links
Jun 2, 2026
Merged

docs: fix Chinese mining guide relative links#6714
Scottcjn merged 1 commit into
Scottcjn:mainfrom
yyswhsccc:bounty-radar/issue-self-mining-guide-links

Conversation

@yyswhsccc
Copy link
Copy Markdown
Contributor

@yyswhsccc yyswhsccc commented Jun 1, 2026

What Changed

  • Fixed three broken relative links in docs/zh-CN/MINING_GUIDE.md.
  • Updated the two top-level document links to resolve from docs/zh-CN/.
  • Updated the troubleshooting FAQ link to resolve to the shared docs FAQ.

Why It Matters

The Chinese mining guide currently sends readers to missing Markdown targets for install prerequisites, CPU antiquity details, and troubleshooting. This keeps the translated guide navigable without changing the surrounding content.

Validation

  • python3 focused Markdown relative-link scan for docs/zh-CN/MINING_GUIDE.md -> checked 6 relative links, missing_count=0.
  • git diff --check upstream/main...HEAD -- docs/zh-CN/MINING_GUIDE.md -> passed.
  • GitHub CI / test is still red with the same repo-wide baseline failure seen on unrelated docs-only PRs; that baseline is being handled separately in test: restore repo-wide CI baseline expectations #6711, without broadening this docs PR.

Scope / Risk

  • Touched files/subsystems: docs/zh-CN/MINING_GUIDE.md
  • Documentation-only change; no runtime behavior, API, protocol, or dependency changes.
  • Related bounty issue: [BOUNTY] Report a Bug — 5-15 RTC #305

wallet: RTC47bc28896a1a4bf240d1fd780f4559b242bcd945

@github-actions github-actions Bot added documentation Improvements or additions to documentation size/XS PR: 1-10 lines labels Jun 1, 2026
@yyswhsccc
Copy link
Copy Markdown
Contributor Author

@Scottcjn This PR is ready for maintainer review.

Validation evidence is listed in the PR body. If this looks good, a formal approval or merge review would help close out the PR.

Copy link
Copy Markdown
Contributor

@jaxint jaxint left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for contributing! 🦀

Code review completed. Nice work! 🚀


Wallet: AhqbFaPBPLMMiaLDzA9WhQcyvv4hMxiteLhPk3NhG1iG

@yyswhsccc
Copy link
Copy Markdown
Contributor Author

@jaxint Thanks for reviewing this. GitHub currently shows this as a comment-only review rather than a formal approval.

Could you re-review when you have a chance? If this looks good, a formal approval would help close out the review.

Copy link
Copy Markdown
Contributor

@jaxint jaxint left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great contribution! 🔍 Reviewed and looks solid.

Copy link
Copy Markdown
Contributor

@jaxint jaxint left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! Thanks for contributing to RustChain. Approved.

Copy link
Copy Markdown
Contributor

@jaxint jaxint left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Well done! This contribution adds real value to the project.

@Scottcjn Scottcjn merged commit 9446359 into Scottcjn:main Jun 2, 2026
11 of 12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation size/XS PR: 1-10 lines

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants