Skip to content

Translate "FAQ: Versioning Policy" #171

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 24 commits into from
Mar 4, 2019
Merged

Translate "FAQ: Versioning Policy" #171

merged 24 commits into from
Mar 4, 2019

Conversation

DmitryMarkov
Copy link
Contributor

Термины, которых нет в руководстве https://github.com/reactjs/ru.reactjs.org/blob/master/TRANSLATION.md я брал с сайта https://semver.org/lang/ru/, на который ссылается статья

@tesseralis tesseralis mentioned this pull request Feb 20, 2019
Closed
88 tasks
@lex111 lex111 added the needs review A pull request ready to be reviewed label Feb 20, 2019
@gcor
Copy link
Collaborator

gcor commented Feb 20, 2019

Мне кажется, что вместо «выпуск» лучше использовать «релиз». Даже гугл-переводчик переводит так дословно. Например, мажорный релиз.

При этом я бы говорил «выпуская» вместо «релизя»

@DmitryMarkov
Copy link
Contributor Author

Мне кажется, что вместо «выпуск» лучше использовать «релиз». Даже гугл-переводчик переводит так дословно. Например, мажорный релиз.

При этом я бы говорил «выпуская» вместо «релизя»

Так и было вначале, но я взял термин с https://semver.org/lang/ru/ на который ссылается статья

@gcor
Copy link
Collaborator

gcor commented Feb 20, 2019

На этом сайте используется слово релиз. Более того, используется фраза «выпуск релизов».

@lex111
Copy link
Member

lex111 commented Feb 21, 2019

На том же сайте используется две формы, но выпуск - для обозначения процесса (выпускаем...), а остальных случаях - релиз. Поэтому давайте аналогично сделаем и здесь.

@DmitryMarkov
Copy link
Contributor Author

@gcor @lex111 заменил "выпуск" на "релиз"

@netlify
Copy link

netlify bot commented Feb 22, 2019 •
edited
Loading

Deploy preview for ru-reactjs ready!

Built with commit c6bec07

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

@lex111
Copy link
Member

lex111 commented Feb 22, 2019

Отлично, ещё стоит поменять "цифру" на "число".

@DmitryMarkov
Copy link
Contributor Author

@lex111 Алексей, я внес правки, но билд не проходит, не подскажете почему?

gcor
gcor reviewed Feb 23, 2019
View reviewed changes
lex111
lex111 requested changes Feb 27, 2019
View reviewed changes
@lex111 lex111 requested a review from another-guy February 27, 2019 23:12
@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 Mar 2, 2019
@another-guy another-guy self-assigned this Mar 2, 2019
gcor
gcor reviewed Mar 2, 2019
View reviewed changes
Copy link
Collaborator

@gcor gcor left a comment

Choose a reason for hiding this comment

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

Пару моментов, которые можно чуть упростить.

@another-guy
Copy link
Collaborator

@lex111 @gcor ваши замечания внесены в перевод? Можно сделать "хвостовую" вычитку?

@DmitryMarkov
Copy link
Contributor Author

@another-guy остались три suggestion, но в них не финальный вариант после обсуждения. Я могу их все объединить в еще один коммит

@another-guy
Copy link
Collaborator

@DmitryMarkov понял. Буду читать тогда скоро. Спасибо!

another-guy
another-guy reviewed Mar 4, 2019
View reviewed changes
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.

Несколько замечаний/предложений по стилю.

Спасибо за перевод, @DmitryMarkov ! 👍

@another-guy another-guy removed their assignment Mar 4, 2019
@lex111 lex111 merged commit 07fe7dd into reactjs:master Mar 4, 2019
@lex111
Copy link
Member

lex111 commented Mar 4, 2019

Спасибо @DmitryMarkov за перевод, а также @gcor и @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 Mar 4, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@gcor gcor gcor left review comments

@another-guy another-guy another-guy left review comments

@lex111 lex111 lex111 approved these changes

Assignees
No one assigned
Labels
needs proofread A read-through is still needed to release the document
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@DmitryMarkov @gcor @lex111 @another-guy