Skip to content

Добавляет "на практике" к js/promise-race#5967

Merged
vitya-ne merged 5 commits intodoka-guide:mainfrom
zheleznikov:js-race-practice
Mar 27, 2026
Merged

Добавляет "на практике" к js/promise-race#5967
vitya-ne merged 5 commits intodoka-guide:mainfrom
zheleznikov:js-race-practice

Conversation

@zheleznikov
Copy link
Copy Markdown
Contributor

Описание

Добавил раздел "На практике" для статьи promise-race
В разделе привел пример использования Promise.race() для создания таймаута при запросе на сервер.

@github-actions github-actions bot added the js Контент по JavaScript label Feb 23, 2026
Comment thread js/promise-race/practice/zheleznikov.md Outdated
@vitya-ne
Copy link
Copy Markdown
Contributor

Привет, спасибо за дополнение!
Посмотри, пожалуйста комент.

@zheleznikov
Copy link
Copy Markdown
Contributor Author

Привет, спасибо за дополнение! Посмотри, пожалуйста комент.

Привет.
Спасибо, что обратил внимание. Действительно формулировка недостаточно корректная. Внес изменения.

Comment thread js/promise-race/practice/zheleznikov.md Outdated
@github-actions
Copy link
Copy Markdown

Превью контента из 1a36781 опубликовано.

@zheleznikov
Copy link
Copy Markdown
Contributor Author

@vitya-ne
Витя, большое спасибо за то, что уделил время ревью. Я обратил внимание на недостатки у себя по стилистике и по точности формулировок (когда знаешь, о чем говоришь, то кажется, что все понятно, а если посмотреть с позиции, что тема еще неизвестна человеку, то нужно исключить возможность неправильно понять).
Постараюсь быть точнее в будущем.

@vitya-ne
Copy link
Copy Markdown
Contributor

@vitya-ne Витя, большое спасибо за то, что уделил время ревью. Я обратил внимание на недостатки у себя по стилистике и по точности формулировок (когда знаешь, о чем говоришь, то кажется, что все понятно, а если посмотреть с позиции, что тема еще неизвестна человеку, то нужно исключить возможность неправильно понять). Постараюсь быть точнее в будущем.

нет проблем )
ждём новых пиаров, удачи

@zheleznikov
Copy link
Copy Markdown
Contributor Author

@vitya-ne Витя, большое спасибо за то, что уделил время ревью. Я обратил внимание на недостатки у себя по стилистике и по точности формулировок (когда знаешь, о чем говоришь, то кажется, что все понятно, а если посмотреть с позиции, что тема еще неизвестна человеку, то нужно исключить возможность неправильно понять). Постараюсь быть точнее в будущем.

нет проблем ) ждём новых пиаров, удачи

Витя, подскажи, пожалуйста, а с моей стороны что-то нужно еще дополнительно или я просто жду пока кто-то с правами владельца апрувнет workflow?

@vitya-ne
Copy link
Copy Markdown
Contributor

@vitya-ne Витя, большое спасибо за то, что уделил время ревью. Я обратил внимание на недостатки у себя по стилистике и по точности формулировок (когда знаешь, о чем говоришь, то кажется, что все понятно, а если посмотреть с позиции, что тема еще неизвестна человеку, то нужно исключить возможность неправильно понять). Постараюсь быть точнее в будущем.

нет проблем ) ждём новых пиаров, удачи

Витя, подскажи, пожалуйста, а с моей стороны что-то нужно еще дополнительно или я просто жду пока кто-то с правами владельца апрувнет workflow?

Нет, для этого пиара ничего не нужно. После доп. ревью и апрува смержится.

Copy link
Copy Markdown
Contributor

@nasty23-star nasty23-star left a comment

Choose a reason for hiding this comment

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

Спасибо за контрибьют! o.。.:*☆

@vitya-ne vitya-ne merged commit 0855d6e into doka-guide:main Mar 27, 2026
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

js Контент по JavaScript

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants