WARNING: THIS SITE IS A MIRROR OF GITHUB.COM / IT CANNOT LOGIN OR REGISTER ACCOUNTS / THE CONTENTS ARE PROVIDED AS-IS / THIS SITE ASSUMES NO RESPONSIBILITY FOR ANY DISPLAYED CONTENT OR LINKS / IF YOU FOUND SOMETHING MAY NOT GOOD FOR EVERYONE, CONTACT ADMIN AT ilovescratch@foxmail.com
Skip to content

Conversation

@Ms2ger
Copy link
Collaborator

@Ms2ger Ms2ger commented Oct 24, 2024

Fixes #1834.

@ADKaster
Copy link

Thanks for drafting this so quickly! I think the comment from 1834 also applies to https://webassembly.github.io/spec/js-api/#instantiate-a-promise-of-a-module

@Ms2ger Ms2ger changed the title [web-api] Use the 'react' algorithm from WebIDL [js-api][web-api] Use the 'react' algorithm from WebIDL Oct 28, 2024
@rossberg
Copy link
Member

@Ms2ger, is this old PR still relevant? If so, who could review it?

@Ms2ger Ms2ger requested a review from eqrion September 11, 2025 09:32
Copy link
Contributor

@eqrion eqrion left a comment

Choose a reason for hiding this comment

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

Promises are 'fun'. LGTM

@Ms2ger
Copy link
Collaborator Author

Ms2ger commented Oct 16, 2025

@rossberg Ryan approved, but doesn't have write access. Could you merge?

@rossberg rossberg merged commit 939d97e into main Oct 16, 2025
12 checks passed
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.

WebAssembly Web API specification should use WebIDL react to promise

5 participants