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

Истправлена опечатка #20

Merged
merged 1 commit into from
Jul 31, 2018

Conversation

LxPvlv
Copy link
Contributor

@LxPvlv LxPvlv commented Jul 31, 2018

Не знаю как делать правки. Подскажите пожалуйста.
На каждую ошибку, опечатку делать пулреквест?
Может делать после прочтения всей главы?

@lex111
Copy link
Member

lex111 commented Jul 31, 2018

@LxPvlv привет!

Это файл readme, он не включается в книгу, но всё равно спасибо за исправление!

На каждую ошибку, опечатку делать пулреквест?

Лучше так не делать...

Может делать после прочтения всей главы?

Это более подходящий вариант.

Спасибо!

@lex111 lex111 merged commit 14ac046 into the-road-to-learn-react:master Jul 31, 2018
@LxPvlv
Copy link
Contributor Author

LxPvlv commented Jul 31, 2018

Только что прочитал contributor.md Там написано:
"Лучше всего это сделать — создать ишью в репозитории перевода."
Так что будет удобней, пулреквест или ишью?

@lex111
Copy link
Member

lex111 commented Jul 31, 2018

@LxPvlv написано не подробно, согласен. Просто не каждый готов тратить время на само исправление, т.е. ишью - это просто оповещение об опечатке, а PR - её исправление. Второе отнимет больше времени, но я дополню текст для большей ясности, спасибо!

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.

None yet

2 participants