Skip to content
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

fix(lazy-compilation): should have separate state in multi-compiler #9545

Merged
merged 1 commit into from
Mar 4, 2025

Conversation

JSerFeng
Copy link
Contributor

@JSerFeng JSerFeng commented Mar 4, 2025

Summary

close #7578
close #8305

Each compiler instance should have its own lazyCompilation state. Implementing communication between compilers and lazyCompilation using a global state can be disrupted by different compilers.

Checklist

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

Sorry, something went wrong.

@github-actions github-actions bot added release: bug fix release: bug related release(mr only) team The issue/pr is created by the member of Rspack. labels Mar 4, 2025
Copy link

netlify bot commented Mar 4, 2025

Deploy Preview for rspack ready!

Name Link
🔨 Latest commit c588e5b
🔍 Latest deploy log https://app.netlify.com/sites/rspack/deploys/67c6e53e2d62900008c6b4ad
😎 Deploy Preview https://deploy-preview-9545--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.

@JSerFeng JSerFeng force-pushed the fix/lazy-compilation-multi-compilerwd branch 3 times, most recently from fada6eb to fee51f6 Compare March 4, 2025 09:27
Copy link

codspeed-hq bot commented Mar 4, 2025

CodSpeed Performance Report

Merging #9545 will not alter performance

Comparing fix/lazy-compilation-multi-compilerwd (c588e5b) with main (fcb9339)

Summary

✅ 7 untouched benchmarks

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature. The key has expired.
@JSerFeng JSerFeng force-pushed the fix/lazy-compilation-multi-compilerwd branch from fee51f6 to c588e5b Compare March 4, 2025 11:34
@JSerFeng JSerFeng marked this pull request as ready for review March 4, 2025 12:52
@JSerFeng JSerFeng merged commit 75f55a2 into main Mar 4, 2025
27 checks passed
@JSerFeng JSerFeng deleted the fix/lazy-compilation-multi-compilerwd branch March 4, 2025 12:52
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) team The issue/pr is created by the member of Rspack.
Projects
None yet
2 participants