-
-
Notifications
You must be signed in to change notification settings - Fork 1.1k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Support ES Module Integration Proposal #4287
Comments
Unfortunately I have absolutely no idea about Deno or the bundler output. I will also have a hard time reviewing something I absolutely have no understanding of. In conclusion: I'm happy to do my best to review fixes to existing targets, but adding a new target is out of the cards unless someone steps up and offer long-term maintenance for that target in |
I renamed the title. This is not Deno specific. It's just standard Wasm imports (which is the future and more ergonomic because the Wasm module is part of the module graph). It feels like something this project would want to support? It would allow dropping a bunch of the outputs in the future. https://github.com/WebAssembly/esm-integration/blob/main/proposals/esm-integration/EXAMPLES.md |
That's definitely in scope. |
Deno 2.1 added support for standard Wasm modules (though there was a bug where it was missing exports like
memory
, though that's now fixed in the latest canary versiondeno upgrade canary
or in the released version if this is being read after November 28th).https://docs.deno.com/runtime/reference/wasm/#wasm-modules
It would be really cool to have support for it in wasm-bindgen because it would mean that the
.wasm
files would be part of the module graph, so remote Deno modules wouldn't need to do orchestration for downloading and caching the wasm files anymore.Since Wasm modules are implemented in a standard way in Deno (or at least should be 😅), this should translate to having support for the browser as well.
Edit: Or I guess this is the "bundler" output? I didn't test yet if it works with imports.
The text was updated successfully, but these errors were encountered: