Skip to content

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

Open
zheleznikov wants to merge 5 commits intodoka-guide:mainfrom
zheleznikov:js-race-practice
Open

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

Conversation

@zheleznikov
Copy link

Описание

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

@github-actions github-actions bot added the js Контент по JavaScript label Feb 23, 2026
@vitya-ne
Copy link
Contributor

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

@zheleznikov
Copy link
Author

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

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

@github-actions
Copy link

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

@zheleznikov
Copy link
Author

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

@vitya-ne
Copy link
Contributor

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

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

@zheleznikov
Copy link
Author

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

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

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

@vitya-ne
Copy link
Contributor

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

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

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

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

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.

3 participants