Skip to content

refactor: use #[napi(module_exports)]#10437

Merged
CPunisher merged 2 commits intoweb-infra-dev:mainfrom
CPunisher:05-21-refactor/module-exports
May 21, 2025
Merged

refactor: use #[napi(module_exports)]#10437
CPunisher merged 2 commits intoweb-infra-dev:mainfrom
CPunisher:05-21-refactor/module-exports

Conversation

@CPunisher
Copy link
Contributor

Summary

#[napi(module_exports)] supports wasm while the legacy #[module_exports] not.

Checklist

  • Tests updated (or not required).
  • Documentation updated (or not required).

@netlify
Copy link

netlify bot commented May 21, 2025

Deploy Preview for rspack canceled.

Built without sensitive environment variables

Name Link
🔨 Latest commit 094c991
🔍 Latest deploy log https://app.netlify.com/projects/rspack/deploys/682d9073bcae1e00088efed9

@codspeed-hq
Copy link

codspeed-hq bot commented May 21, 2025

CodSpeed Performance Report

Merging #10437 will not alter performance

Comparing CPunisher:05-21-refactor/module-exports (094c991) with main (3c46a52)

🎉 Hooray! codspeed-node just leveled up to 4.0.1!

A heads-up, this is a breaking change and it might affect your current performance baseline a bit. But here's the exciting part - it's packed with new, cool features and promises improved result stability 🥳!
Curious about what's new? Visit our releases page to delve into all the awesome details about this new version.

Summary

✅ 11 untouched benchmarks

@CPunisher CPunisher merged commit 8cf1df4 into web-infra-dev:main May 21, 2025
35 checks passed
@CPunisher CPunisher deleted the 05-21-refactor/module-exports branch May 21, 2025 09:57
@CPunisher CPunisher mentioned this pull request May 21, 2025
8 tasks
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.

2 participants