Skip to content

Translate Test Renderer to Russian #240

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 33 commits into from
Mar 13, 2019
Merged

Translate Test Renderer to Russian #240

merged 33 commits into from
Mar 13, 2019

Conversation

theghostbel
Copy link
Contributor

No description provided.

@theghostbel theghostbel mentioned this pull request Mar 8, 2019
Closed
88 tasks
@netlify
Copy link

netlify bot commented Mar 8, 2019 •
edited
Loading

Deploy preview for ru-reactjs ready!

Built with commit 04b8f67

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

@ntishkevich ntishkevich added the needs review A pull request ready to be reviewed label Mar 9, 2019
ntishkevich
ntishkevich reviewed Mar 9, 2019
View reviewed changes
Copy link
Collaborator

@ntishkevich ntishkevich left a comment

Choose a reason for hiding this comment

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

Для первой итерации ревью изменений хватит. Пересмотрите diff, так как были добавлены лишние пустые строки, также остался оригинальный текст, который упустили видимо.

lex111
lex111 reviewed Mar 10, 2019
View reviewed changes
Copy link
Member

@lex111 lex111 left a comment •
edited
Loading

Choose a reason for hiding this comment

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

theghostbel:patch-1

Плохая идея переводить с гитхаба, наверное из-за много пустых строк, из-за которых сложно сравнивать перевод и оригинальный текст :(

Давайте примем предложения, которые есть, потом исправим пустые строчки.

ntishkevich and others added 8 commits March 10, 2019 09:52
@theghostbel
Copy link
Contributor Author

@lex111, @ntishkevich, спасибо за проверку!

Применил, где возможно, предложенные правки, удалил лишние пустые строки и пр. опечатки.

Два абзаца для меня всё еще остаются проблемными - про assertions и фиктивные рефы...

@ntishkevich
Copy link
Collaborator

@lex111, @ntishkevich, спасибо за проверку!

Применил, где возможно, предложенные правки, удалил лишние пустые строки и пр. опечатки.

Два абзаца для меня всё еще остаются проблемными - про assertions и фиктивные рефы...

Хорошо, попробую вычитать эти два абзаца. Также могли бы перевести еще название этой статьи в файле nav.yml? Спасибо.

@gcor
Copy link
Collaborator

gcor commented Mar 12, 2019

@another-guy, это последняя статья для 100% core-переводов 🔥

@theghostbel
Copy link
Contributor Author

theghostbel commented Mar 12, 2019 •
edited
Loading

@ntishkevich обновил ссылку в nav.yml

another-guy
another-guy requested changes Mar 12, 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.

Поправки по мелочам. Спасибо за перевод @theghostbel !

@another-guy
Copy link
Collaborator

@gcor да, даже не верится! Контрибьюторы выполнили большую работу в достаточно короткий срок! Мало какие другие языки уже закончили переводы и имеют столь же полное покрытие тем!

another-guy and others added 18 commits March 13, 2019 11:15
@another-guy @theghostbel
Вычитка - как jest делает снимки
f5f6181 
Co-Authored-By: theghostbel <[email protected]>
@another-guy @theghostbel
Вычитка - "простые JS-объекты" -> "обычные JS-объекты"
67d9fd0 
Co-Authored-By: theghostbel <[email protected]>
@another-guy @theghostbel
Вычитка - запятая
06bcf29 
Co-Authored-By: theghostbel <[email protected]>
@another-guy @theghostbel
Вычитка - "создание экземпляра" -> "создает экземпляр"
04fe55b 
Co-Authored-By: theghostbel <[email protected]>
@another-guy @theghostbel
Вычитка - ссылка на русскую документацию тестирования снимками в Jest
f924024 
Co-Authored-By: theghostbel <[email protected]>
@another-guy @theghostbel
Вычитка - генерация ошибки вместо возврата
7c8261a 
Co-Authored-By: theghostbel <[email protected]>
@another-guy @theghostbel
Вычитка - "самописные" - они же "пользовательские"
9aca18f 
Co-Authored-By: theghostbel <[email protected]>
@another-guy @theghostbel
Вычитка - тип type
2aa9783 
Co-Authored-By: theghostbel <[email protected]>
@another-guy @theghostbel
Вычитка - долой деепричастия
0c46ec2 
Co-Authored-By: theghostbel <[email protected]>
@another-guy @theghostbel
Вычитка - тут нужно деепричастие
0d7cc33 
Co-Authored-By: theghostbel <[email protected]>
@another-guy @theghostbel
Вычитка - исправить опечатку
c51b170 
Co-Authored-By: theghostbel <[email protected]>
@another-guy @theghostbel
Вычитка - пропсы props
f1c92f5 
Co-Authored-By: theghostbel <[email protected]>
@another-guy @theghostbel
Вычитка - unmount -> отмонтировать -> размонтировать -> демонтировать
84acb10 
Co-Authored-By: theghostbel <[email protected]>
@another-guy @theghostbel
Вычитка - перестановка приложения для легкого чтения
51aef50 
Co-Authored-By: theghostbel <[email protected]>
@another-guy @theghostbel
Вычитка - короче и легче читать
f3b6d9e 
Co-Authored-By: theghostbel <[email protected]>
@another-guy @theghostbel
Вычитка - опять про ошибки
93e5ca4 
Co-Authored-By: theghostbel <[email protected]>
@theghostbel
Update reference-test-renderer.md
daca115 
Опечаточка - букву Б забыли
@another-guy
Update reference-test-renderer.md
04b8f67
@another-guy another-guy merged commit a335ae0 into reactjs:master Mar 13, 2019
@another-guy
Copy link
Collaborator

🎉 🎉 🎉

@another-guy another-guy added needs proofread A read-through is still needed to release the document and removed needs review A pull request ready to be reviewed labels Mar 13, 2019
@theghostbel
Copy link
Contributor Author

@another-guy Спасибо за вычитку!
Также спасибо @lex111, @gcor, @ntishkevich за помощь с проверкой и советами!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@lex111 lex111 lex111 left review comments

@gcor gcor gcor left review comments

@ntishkevich ntishkevich ntishkevich left review comments

@another-guy another-guy another-guy 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.
5 participants
@theghostbel @ntishkevich @gcor @another-guy @lex111