Skip to content
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

Merged
merged 43 commits into from
Feb 27, 2019

Conversation

AlLevykin
Copy link
Contributor

No description provided.

@lex111 lex111 added the needs review A pull request ready to be reviewed label Feb 15, 2019
content/docs/how-to-contribute.md Outdated Show resolved Hide resolved
content/docs/how-to-contribute.md Outdated Show resolved Hide resolved
gcor and others added 3 commits February 16, 2019 01:14
Co-Authored-By: AlLevykin <a.a.levykin@gmail.com>
Co-Authored-By: AlLevykin <a.a.levykin@gmail.com>
content/docs/how-to-contribute.md Outdated Show resolved Hide resolved
content/docs/how-to-contribute.md Outdated Show resolved Hide resolved
content/docs/how-to-contribute.md Outdated Show resolved Hide resolved
content/docs/how-to-contribute.md Outdated Show resolved Hide resolved
ntishkevich and others added 3 commits February 16, 2019 15:20
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>
@tesseralis tesseralis mentioned this pull request Feb 16, 2019
88 tasks
Copy link
Contributor

@angryermine angryermine left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Привет! Предлагаю не изменять количество букв в слове pull, раз уж мы используем транслитерацию.

content/docs/how-to-contribute.md Outdated Show resolved Hide resolved
content/docs/how-to-contribute.md Outdated Show resolved Hide resolved
content/docs/how-to-contribute.md Outdated Show resolved Hide resolved
content/docs/how-to-contribute.md Outdated Show resolved Hide resolved
content/docs/how-to-contribute.md Outdated Show resolved Hide resolved
content/docs/how-to-contribute.md Outdated Show resolved Hide resolved
content/docs/how-to-contribute.md Outdated Show resolved Hide resolved
content/docs/how-to-contribute.md Outdated Show resolved Hide resolved
angryermine and others added 3 commits February 17, 2019 16:23
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>
content/docs/how-to-contribute.md Outdated Show resolved Hide resolved
content/docs/how-to-contribute.md Outdated Show resolved Hide resolved
content/docs/how-to-contribute.md Outdated Show resolved Hide resolved
@lex111
Copy link
Member

lex111 commented Feb 17, 2019

@lfedorov мы используем словарь Веб-стандартов, и там pull request переводится с одной "л"

@angryermine
Copy link
Contributor

Прошу прощения, почему-то пропустил этот момент

@AlLevykin
Copy link
Contributor Author

Ну и Гитхаб аналогично https://github.com/web-standards-ru/dictionary/blob/master/git.md#g

@lex111
Copy link
Member

lex111 commented Feb 17, 2019

@AlLevykin не совсем так, поскольку мы для этого ввели правило:

Не переводите наименования компаний (Facebook, Airbnb), библиотек (Jest, Enzyme) и т.д.

Название ветки тоже не нужно переводить.

@angryermine
Copy link
Contributor

Почистил ревью от лишних изменений.

angryermine and others added 6 commits February 17, 2019 17:46
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>
@netlify
Copy link

netlify bot commented Feb 24, 2019

Deploy preview for ru-reactjs ready!

Built with commit 3e7482e

https://deploy-preview-149--ru-reactjs.netlify.com

Copy link
Collaborator

@another-guy another-guy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Есть кое-какие советы по улучшению. В целом, уже читается очень неплохо ! Спасибо, @AlLevykin 👍

content/docs/how-to-contribute.md Outdated Show resolved Hide resolved
content/docs/how-to-contribute.md Outdated Show resolved Hide resolved
content/docs/how-to-contribute.md Outdated Show resolved Hide resolved
content/docs/how-to-contribute.md Outdated Show resolved Hide resolved
content/docs/how-to-contribute.md Outdated Show resolved Hide resolved
content/docs/how-to-contribute.md Outdated Show resolved Hide resolved
content/docs/how-to-contribute.md Outdated Show resolved Hide resolved
content/docs/how-to-contribute.md Outdated Show resolved Hide resolved
content/docs/how-to-contribute.md Outdated Show resolved Hide resolved
content/docs/how-to-contribute.md Outdated Show resolved Hide resolved
@another-guy another-guy added work in progress Author opened the pull-request but it's still being worked on and removed needs review A pull request ready to be reviewed labels Feb 25, 2019
another-guy and others added 14 commits February 25, 2019 18:45
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>
@another-guy
Copy link
Collaborator

@AlLevykin исправь, пожалуйста, merge conflict. Спасибо за внесение изменений.

Copy link
Member

@lex111 lex111 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Внёс непосредственно свои правки, если что исправим/улучшим в отдельных в PR.

@lex111 lex111 merged commit 8161fb8 into reactjs:master Feb 27, 2019
@lex111
Copy link
Member

lex111 commented Feb 27, 2019

Спасибо @AlLevykin, а также @lfedorov, @gcor, @ntishkevich и @another-guy за ревью! 🎉

@lex111 lex111 added needs proofread A read-through is still needed to release the document and removed work in progress Author opened the pull-request but it's still being worked on labels Feb 27, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
needs proofread A read-through is still needed to release the document
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants