Skip to content

Conversation

kimorkim
Copy link
Collaborator

No description provided.

@github-actions github-actions bot added the issue 내부 이슈 처리를 위한 PR label Jul 23, 2024
@leunge leunge merged commit 2f5d3c7 into develop Jul 23, 2024
@leunge leunge deleted the issue/8447-2 branch July 23, 2024 08:49
static funcAsyncExecute = async (funcCode, funcExecutor, _promises = []) => {
await Promise.all(_promises);
if (Entry.engine.isState('pause')) {
return this.funcAsyncExecute(funcCode, funcExecutor, _promises);
Copy link
Collaborator

Choose a reason for hiding this comment

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

@kimorkim
무한대기가 끝난 이후에 다시 funcAsyncExecute을 실행하는 부분은 필요가 없는 건가요?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

@Tnks2U
해당코드가 그 밑의

return new Promise((resolve, reject) => {

부분을 호출하는 것과 동일한것으로 여겨서 제외했는데. 이제 보니 run 체크 하는걸 한번더 하는게 나아 보이네요. 살려 두는게 좀더 실용적 이겠네요.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
issue 내부 이슈 처리를 위한 PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants