docs: add Korean (ko-KR) and Arabic (ar-SA) translations#6775
docs: add Korean (ko-KR) and Arabic (ar-SA) translations#6775JesusMP22 wants to merge 5 commits into
Conversation
|
Welcome to RustChain! Thanks for your first pull request. Before we review, please make sure:
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! |
luisalias007-cmyk
left a comment
There was a problem hiding this comment.
Reviewed the PR scope and new files. The translation additions are mixed with a new FreeBSD miner installer, so this needs either scope separation or the installer must satisfy the repo requirements before merge.
| @@ -0,0 +1,152 @@ | |||
| #!/bin/bash | |||
There was a problem hiding this comment.
This PR is titled/described as adding Korean and Arabic README translations, but it also introduces a new executable installer. Since this is a new code/script file, it should include the repo-required SPDX license header before merge, and ideally be split into its own focused PR because it is unrelated to the translation scope.
jaxint
left a comment
There was a problem hiding this comment.
Great contribution! 🔍 Reviewed and looks solid.
jaxint
left a comment
There was a problem hiding this comment.
LGTM! Thanks for contributing to RustChain. Approved.
Code Review for PR #6775Files reviewed: 10 files (+1234/-0) Files examined:
General observations:
Assessment:
Recommendation: Looks good to merge. Wallet for bounty: jesusmp Claiming code review bounty. Review completed on all 10 changed files. |
jaxint
left a comment
There was a problem hiding this comment.
Great contribution! This PR looks well-structured and follows best practices.
|
Closing as superseded by #6776, which is the superset of this translation batch (same languages plus more). Consolidating to one PR keeps review clean — thanks @JesusMP22. |
Documentation Translations
Added README translations for:
Wallet: jesusmp