Skip to content

fix: napi register module twice#9648

Merged
stormslowly merged 5 commits intomainfrom
chore/merge_binding
Mar 18, 2025
Merged

fix: napi register module twice#9648
stormslowly merged 5 commits intomainfrom
chore/merge_binding

Conversation

@stormslowly
Copy link
Contributor

@stormslowly stormslowly commented Mar 12, 2025

Summary

we can not forbid a dep crate with napi of another napi craete in the same workspace don't registering exports.
So move the node fs create back to binding.

Checklist

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

@github-actions github-actions bot added the release: bug fix release: bug related release(mr only) label Mar 12, 2025
@netlify
Copy link

netlify bot commented Mar 12, 2025

Deploy Preview for rspack ready!

Name Link
🔨 Latest commit 89ee421
🔍 Latest deploy log https://app.netlify.com/sites/rspack/deploys/67d8e7dc82ad8a0008bfeb35
😎 Deploy Preview https://deploy-preview-9648--rspack.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@codspeed-hq
Copy link

codspeed-hq bot commented Mar 14, 2025

CodSpeed Performance Report

Merging #9648 will not alter performance

Comparing chore/merge_binding (89ee421) with main (7cd8802)

🎉 Hooray! codspeed-rust just leveled up to 2.7.2!

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

✅ 8 untouched benchmarks

@stormslowly stormslowly enabled auto-merge (squash) March 18, 2025 03:27
@stormslowly stormslowly merged commit a2d03e8 into main Mar 18, 2025
56 of 57 checks passed
@stormslowly stormslowly deleted the chore/merge_binding branch March 18, 2025 07:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

release: bug fix release: bug related release(mr only)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants