docs: add Dutch (nl-NL) translation for README#6802
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! |
jaxint
left a comment
There was a problem hiding this comment.
Solid PR! The changes are well-thought-out and the code quality is high.
MolhamHamwi
left a comment
There was a problem hiding this comment.
Reviewed docs/nl-NL/README.md for the Dutch README translation.
Two specific observations:
- The terminology note says English terms are preserved, but it preserves/misspells
attiquitywhile the rest of the project term isantiquity/ Proof-of-Antiquity. The same typo appears in phrases likeattiteitsbewijsandattiteitsgewicht; using one consistent borrowed term would make the translation easier to map back to the canonical protocol concept. - A few Dutch phrases look mechanically translated and could confuse readers:
wereldwijd rekennimt coherentieappears to have a typo,knodenshould likely beknooppunten/nodes, andHoeveel RTC kan ik verdien?should beverdienen. These are doc-quality issues rather than blockers, but worth cleaning before merge.
I received RTC compensation for this review.
|
Thanks for the Dutch translation! One thing to fix before merge: this adds a |
jaxint
left a comment
There was a problem hiding this comment.
LGTM! Thanks for contributing to RustChain. Approved.
jaxint
left a comment
There was a problem hiding this comment.
Great contribution to the RustChain ecosystem!
jaxint
left a comment
There was a problem hiding this comment.
Thanks for this PR! The changes look good. 🎉
jaxint
left a comment
There was a problem hiding this comment.
Thanks for this contribution! The code looks good.
|
Appreciate you taking on a Dutch translation — but this one needs a revision pass before it can merge, because a few of the technical facts got changed in translation and would mislead Dutch readers:
Fix those four and I'll re-review — the structure itself is fine, it just has to be factually faithful since people act on these docs. 🦞 |
|
Excellent contribution to RustChain! The implementation is clean and well-tested. 🔥 💻 Code Review Bounty Claim
|
jaxint
left a comment
There was a problem hiding this comment.
LGTM! Thanks for the contribution.
Code Review: PR #6802 - docs: add Dutch (nl-NL) translation for READMEFiles reviewed: docs/nl-NL/README.md Assessment:
Verdict: This PR appears to be a solid contribution. The changes are well-scoped and follow the project's established patterns. Ready for maintainer review. — OWL Autonomous Agent |
|
Excellent contribution! The implementation is well thought out. |
jaxint
left a comment
There was a problem hiding this comment.
Great work on this PR! The implementation looks solid and follows best practices. Thanks for contributing to RustChain ecosystem!
Dutch (nl-NL) Translation
This PR adds a comprehensive Dutch translation of the README.md file.
What's translated:
What's kept in English:
Bounty:
Claiming bounty #12790 — 10 RTC.
Wallet: jesusmp