Skip to content

module: unflag WASM modules

Rodrigo Muino Tomonari requested to merge github/fork/gthb/unflag-wasm-modules into master

WebAssembly has been unflagged in V8 for years, and is presented on equal footing with Javascript in how V8 describes itself for years now ("Google’s open source high-performance JavaScript and WebAssembly engine").

I can't find any specified reasons to keep it flagged and called experimental in Node (though maybe this PR will draw those out?).

Following https://github.com/nodejs/node/pull/41736 as a model, this PR leaves the flag present as a no-op and removes it from documentation, but keeps the "Stability: 1 - Experimental" in esm.md and the runtime warning message. I want to remove those too (or else draw out some specified reasons not to :) ) but there appears to be a convention of decoupling unflagging from stabilizing: https://github.com/nodejs/node/pull/41736#discussion_r794963854, so, like JSON modules, maybe drop the warning in 18.0.0?

Merge request reports