Skip to content

Update community.md #5

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

Merged
merged 2 commits into from
Sep 23, 2016
Merged

Conversation

ValentinNikolaev
Copy link

@ValentinNikolaev ValentinNikolaev commented Sep 23, 2016

This change is Reviewable

@mkpankov
Copy link
Member

Спасибо за правки! Оставил несколько комментариев.


Reviewed 1 of 1 files at r1.
Review status: all files reviewed at latest revision, 8 unresolved discussions.


ru-RU/community.md, line 8 at r1 (raw file):

преимущество

преимуществом


ru-RU/community.md, line 43 at r1 (raw file):

а также,

запятая не нужна


ru-RU/community.md, line 61 at r1 (raw file):

для поиска помощи

По-моему, это как-то не по-русски. Я бы написал

чтобы попросить о помощи


ru-RU/community.md, line 68 at r1 (raw file):
Думаю, стоит добавить "канал"

канал обо всём, что связано с Rust

Также обратите внимание на "связано"


ru-RU/community.md, line 89 at r1 (raw file):
Что-то я не уверен, что это правильно. В оригинале шла речь о том, что сама команда разработки документации (официальной) является неофициальной. Теперь же команда документации уже официальна. Так что оригинал устарел и правильно будет

канал разработчиков документации


ru-RU/community.md, line 156 at r1 (raw file):

* [Основная команда][core_team] отвечает за направление дизайна
  и разработки. Контролирует введение новых возможностей, а также

Здесь было лучше по-старому. Сейчас эта точка сильно рвёт однородный ряд.


ru-RU/community.md, line 176 at r1 (raw file):

общественные связи

Тогда уж

связи с общественностью


ru-RU/community.md, line 214 at r1 (raw file):

библиотеки

библиотеках


Comments from Reviewable

@ValentinNikolaev
Copy link
Author

Спасибо. Чуть позже все исправлю

@ValentinNikolaev
Copy link
Author

Исправил

@ValentinNikolaev
Copy link
Author

Review status: 0 of 1 files reviewed at latest revision, 8 unresolved discussions.


ru-RU/community.md, line 8 at r1 (raw file):

Previously, mkpankov (Michael Pankov) wrote…

преимущество

преимуществом

Done.

ru-RU/community.md, line 43 at r1 (raw file):

Previously, mkpankov (Michael Pankov) wrote…

а также,

запятая не нужна

Done.

ru-RU/community.md, line 61 at r1 (raw file):

Previously, mkpankov (Michael Pankov) wrote…

для поиска помощи

По-моему, это как-то не по-русски. Я бы написал

чтобы попросить о помощи

Done.

ru-RU/community.md, line 68 at r1 (raw file):

Previously, mkpankov (Michael Pankov) wrote…

Думаю, стоит добавить "канал"

канал обо всём, что связано с Rust

Также обратите внимание на "связано"

Done.

ru-RU/community.md, line 89 at r1 (raw file):

Previously, mkpankov (Michael Pankov) wrote…

Что-то я не уверен, что это правильно. В оригинале шла речь о том, что сама команда разработки документации (официальной) является неофициальной. Теперь же команда документации уже официальна. Так что оригинал устарел и правильно будет

канал разработчиков документации

Done.

ru-RU/community.md, line 156 at r1 (raw file):

Previously, mkpankov (Michael Pankov) wrote…

Здесь было лучше по-старому. Сейчас эта точка сильно рвёт однородный ряд.

Done.

ru-RU/community.md, line 214 at r1 (raw file):

Previously, mkpankov (Michael Pankov) wrote…

библиотеки

библиотеках

Done.

Comments from Reviewable

@ValentinNikolaev
Copy link
Author

Review status: 0 of 1 files reviewed at latest revision, 8 unresolved discussions.


ru-RU/community.md, line 176 at r1 (raw file):

Previously, mkpankov (Michael Pankov) wrote…

общественные связи

Тогда уж

связи с общественностью

Done.

Comments from Reviewable

@mkpankov
Copy link
Member

👍


Reviewed 1 of 1 files at r2.
Review status: all files reviewed at latest revision, all discussions resolved.


Comments from Reviewable

@mkpankov mkpankov merged commit 0e13f93 into rust-lang-ru:master Sep 23, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants