Skip to content

Comments

chore: Do not use Promise.all.#34

Merged
mcollina merged 1 commit intomainfrom
no-promise-all
Feb 20, 2026
Merged

chore: Do not use Promise.all.#34
mcollina merged 1 commit intomainfrom
no-promise-all

Conversation

@ShogunPanda
Copy link
Contributor

Fixes #21

Signed-off-by: Paolo Insogna <paolo@cowtech.it>
@ShogunPanda ShogunPanda requested a review from mcollina February 20, 2026 10:40
Copy link
Member

@mcollina mcollina left a comment

Choose a reason for hiding this comment

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

lgtm

@mcollina mcollina merged commit cfb99e1 into main Feb 20, 2026
1 check passed
@mcollina mcollina deleted the no-promise-all branch February 20, 2026 10:45
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.

getJobStates should not call getJobState with Promise.all

2 participants