Conversation
|
Error: the article already has PR number in the Progress Issue #1, it's 77 |
|
|
|
@Arthur-Gyulabyan thanks for the PR. I'll review it soon. Keep up the good work! |
|
@bugron thank you for response. |
|
This looks great. I'll be reviewing your translations today. For future let's keep the one article - one PR principle. |
|
Oh, sorry for that mistake. I'll follow the principle in further PRs. |
bugron
left a comment
There was a problem hiding this comment.
@Arthur-Gyulabyan thanks again for the PR. I am amazed by the quality of the translation, really!
Please do not get discouraged by the number of comments! Except for those new line comments there are just a few things to address and we'll be ready to merge this.
Way to go!
|
Please make the requested changes. After it, add a comment "/done". |
|
@bugron thank you for review! The issue with new lines occurred because of my editor's settings. I fixed it, so now everything is okay. I also fixed typos and other bugs. |
|
/done |
|
Thanks for your first PR! Merging and looking forward to your future contributions. 🎉🎉🎉 |
|
Thank you 💖 I updated the Progress Issue #1 🎉 🎉 🎉 |
|
Error: the article already translated in the Progress Issue #1 |
Data types section translated.