-
Notifications
You must be signed in to change notification settings - Fork 388
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Translate "How to Contribute" (how-to-contribute.md) #149
Translate "How to Contribute" (how-to-contribute.md) #149
Conversation
Co-Authored-By: AlLevykin <a.a.levykin@gmail.com>
Co-Authored-By: AlLevykin <a.a.levykin@gmail.com>
Co-Authored-By: AlLevykin <a.a.levykin@gmail.com>
Co-Authored-By: AlLevykin <a.a.levykin@gmail.com>
Co-Authored-By: AlLevykin <a.a.levykin@gmail.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Привет! Предлагаю не изменять количество букв в слове pull, раз уж мы используем транслитерацию.
Co-Authored-By: AlLevykin <a.a.levykin@gmail.com>
Co-Authored-By: AlLevykin <a.a.levykin@gmail.com>
Co-Authored-By: AlLevykin <a.a.levykin@gmail.com>
@lfedorov мы используем словарь Веб-стандартов, и там pull request переводится с одной "л" |
Прошу прощения, почему-то пропустил этот момент |
Ну и Гитхаб аналогично https://github.com/web-standards-ru/dictionary/blob/master/git.md#g |
@AlLevykin не совсем так, поскольку мы для этого ввели правило:
Название ветки тоже не нужно переводить. |
Почистил ревью от лишних изменений. |
Co-Authored-By: AlLevykin <a.a.levykin@gmail.com>
Co-Authored-By: AlLevykin <a.a.levykin@gmail.com>
Co-Authored-By: AlLevykin <a.a.levykin@gmail.com>
Co-Authored-By: AlLevykin <a.a.levykin@gmail.com>
Co-Authored-By: AlLevykin <a.a.levykin@gmail.com>
Co-Authored-By: AlLevykin <a.a.levykin@gmail.com>
Deploy preview for ru-reactjs ready! Built with commit 3e7482e |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Есть кое-какие советы по улучшению. В целом, уже читается очень неплохо ! Спасибо, @AlLevykin 👍
Co-Authored-By: AlLevykin <a.a.levykin@gmail.com>
Co-Authored-By: AlLevykin <a.a.levykin@gmail.com>
Co-Authored-By: AlLevykin <a.a.levykin@gmail.com>
Co-Authored-By: AlLevykin <a.a.levykin@gmail.com>
Co-Authored-By: AlLevykin <a.a.levykin@gmail.com>
Co-Authored-By: AlLevykin <a.a.levykin@gmail.com>
Co-Authored-By: AlLevykin <a.a.levykin@gmail.com>
Co-Authored-By: AlLevykin <a.a.levykin@gmail.com>
Co-Authored-By: AlLevykin <a.a.levykin@gmail.com>
Co-Authored-By: AlLevykin <a.a.levykin@gmail.com>
This reverts commit b1c134e.
@AlLevykin исправь, пожалуйста, merge conflict. Спасибо за внесение изменений. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Внёс непосредственно свои правки, если что исправим/улучшим в отдельных в PR.
Спасибо @AlLevykin, а также @lfedorov, @gcor, @ntishkevich и @another-guy за ревью! 🎉 |
No description provided.