BigInt#120
Conversation
|
Error: the article already has PR number in the Progress Issue #1, it's 117 |
There was a problem hiding this comment.
You don't need to make a new pull request for any changes you make. you can add more commits by pushing to the BigInt branch on your fork.
In addition to that, please read the Translation Tips:
Please keep line breaks and paragraphs "as is": don't add new lines and don't remove existing ones. Makes it easy to merge future changes from the English version into the translation.
You've removed the newline on line 8, please add it back.
|
Please make the requested changes. After it, add a comment "/done". |
|
Thanks for the review, @mahdyar |
1-js/99-js-misc/05-bigint/article.md
Outdated
There was a problem hiding this comment.
Math operators means عملگرهای ریاضی, not عملیات ریاضی.
1-js/99-js-misc/05-bigint/article.md
Outdated
There was a problem hiding this comment.
Put 5/2 between backticks as it was.
1-js/99-js-misc/05-bigint/article.md
Outdated
There was a problem hiding this comment.
There is no need to translate Error here.
1-js/99-js-misc/05-bigint/article.md
Outdated
1-js/99-js-misc/05-bigint/article.md
Outdated
There was a problem hiding this comment.
مقایسهای would be a better translation.
1-js/99-js-misc/05-bigint/article.md
Outdated
There was a problem hiding this comment.
There is no ابتدا in the text, don't add your own words. Also, you missed so.
پس باید از...
1-js/99-js-misc/05-bigint/article.md
Outdated
|
Please make the requested changes. After it, add a comment "/done". |
|
Thanks, |
3 participants
Translated the BigInt page.