Skip to content

docs: add translations index to README header#6776

Open
JesusMP22 wants to merge 6 commits into
Scottcjn:mainfrom
JesusMP22:docs-readme-improvements
Open

docs: add translations index to README header#6776
JesusMP22 wants to merge 6 commits into
Scottcjn:mainfrom
JesusMP22:docs-readme-improvements

Conversation

@JesusMP22
Copy link
Copy Markdown
Contributor

@JesusMP22 JesusMP22 commented Jun 1, 2026

Summary

Replaced the single Chinese入口 link with a full translations index linking to all 13 available translations:
en, pt-BR, zh-CN, zh-TW, vi, de-DE, es, hi, ja, ru, it, fr, ko, ar

Wallet: jesusmp

@github-actions github-actions Bot added documentation Improvements or additions to documentation BCOS-L1 Beacon Certified Open Source tier BCOS-L1 (required for non-doc PRs) labels Jun 1, 2026
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Jun 1, 2026

Welcome to RustChain! Thanks for your first pull request.

Before we review, please make sure:

  • Non-doc PRs have a BCOS-L1 or BCOS-L2 label
  • Doc-only PRs are exempt from BCOS tier labels when they only touch docs/**, *.md, or common image/PDF files
  • New code files include an SPDX license header
  • You've tested your changes against the live node

Bounty tiers: Micro (1-10 RTC) | Standard (20-50) | Major (75-100) | Critical (100-150)

A maintainer will review your PR soon. Thanks for contributing!

@github-actions github-actions Bot added the size/M PR: 51-200 lines label Jun 1, 2026
luisalias007-cmyk
luisalias007-cmyk suggested changes Jun 1, 2026
View reviewed changes
Copy link
Copy Markdown
Contributor

@luisalias007-cmyk luisalias007-cmyk left a comment

Choose a reason for hiding this comment

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

Reviewed the translation index against the PR head. The new table is useful, but one link is currently broken and should be fixed before merge.

Comment thread README.md

中文入口: [中文文档](docs/zh-CN/README.md) · [中文 API 快速参考](docs/zh-CN/API.md)
#### 🌍 Translations
[English](README.md) · [Português](README.pt-BR.md) · [中文](README.zh.md) · [繁體中文](README_ZH-TW.md) · [Tiếng Việt](README.vi.md) · [Deutsch](docs/de-DE/README.md) · [Español](README_ES.md) · [हिन्दी](README_HI.md) · [日本語](README_JA.md) · [Русский](README_RU.md) · [Italiano](docs/it-IT/README.md) · [Français](docs/fr-FR/README.md) · [한국어](docs/ko-KR/README.md) · [العربية](docs/ar-SA/README.md)
Copy link
Copy Markdown
Contributor

@luisalias007-cmyk luisalias007-cmyk Jun 1, 2026

Choose a reason for hiding this comment

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

This Chinese link points to README.zh.md, but that file does not exist at this PR head. I checked https://raw.githubusercontent.com/Scottcjn/Rustchain/32dc2c9e48a4fab44cf1f947168021e24fc44b4c/README.zh.md and it returns 404. The existing Chinese entry on main uses docs/zh-CN/README.md, so this should keep pointing there or add the missing root-level file before merging.

@luisalias007-cmyk luisalias007-cmyk mentioned this pull request Jun 1, 2026
Open
jaxint
jaxint reviewed Jun 2, 2026
View reviewed changes
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.

jaxint
jaxint approved these changes Jun 2, 2026
View reviewed changes
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.

jaxint
jaxint reviewed Jun 2, 2026
View reviewed changes
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 the contribution! Good use of Rust idioms throughout.

This was referenced Jun 2, 2026
Closed
Closed
@Scottcjn
Copy link
Copy Markdown
Owner

Scottcjn commented Jun 2, 2026

Thanks @JesusMP22 — happy to land these translations (closed #6774/#6775 as subsets of this one). One rebase needed: this PR re-adds install-miner-freebsd.sh and miners/freebsd/README.md, which already merged via #6753. Please rebase onto current main and drop those two files so the PR is translations-only (the docs/* + README* + i18n changes). Then I'll merge — the README language index (#6795) is already live and pointing at these.

jaxint
jaxint approved these changes Jun 2, 2026
View reviewed changes
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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

2 more reviewers

@luisalias007-cmyk luisalias007-cmyk luisalias007-cmyk requested changes

@jaxint jaxint jaxint approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

BCOS-L1 Beacon Certified Open Source tier BCOS-L1 (required for non-doc PRs) documentation Improvements or additions to documentation size/M PR: 51-200 lines

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@JesusMP22 @Scottcjn @jaxint @luisalias007-cmyk