Skip to content

fix: should union available modules when depend on multiple entry #10342

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

Merged
merged 1 commit into from
Jul 1, 2025

Conversation

JSerFeng
Copy link
Contributor

Summary

close #10261

Available modules should be intersection for async chunks' parents, and should be union for entries' parents.

Using dependOn will make the entry loading after all parents loaded, so all modules in parents are available at that time, should union their available modules.

Checklist

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

Copy link

netlify bot commented May 13, 2025

Deploy Preview for rspack canceled.

Name Link
🔨 Latest commit 3495729
🔍 Latest deploy log https://app.netlify.com/projects/rspack/deploys/6863be044bb6570008e41d34

@github-actions github-actions bot added team The issue/pr is created by the member of Rspack. release: bug fix release: bug related release(mr only) labels May 13, 2025
@JSerFeng JSerFeng force-pushed the fix/depend-on-multiple branch from f01bad9 to 57dd183 Compare May 26, 2025 04:00
@JSerFeng JSerFeng force-pushed the fix/depend-on-multiple branch from 57dd183 to c4fca73 Compare July 1, 2025 10:03
Copy link

codspeed-hq bot commented Jul 1, 2025

CodSpeed Performance Report

Merging #10342 will not alter performance

Comparing fix/depend-on-multiple (3495729) with main (fc9cb1e)

Summary

✅ 16 untouched benchmarks

@JSerFeng JSerFeng force-pushed the fix/depend-on-multiple branch from c4fca73 to 3495729 Compare July 1, 2025 10:52
@JSerFeng JSerFeng merged commit 2e1edb9 into main Jul 1, 2025
40 checks passed
@JSerFeng JSerFeng deleted the fix/depend-on-multiple branch July 1, 2025 11:28
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
Development

Successfully merging this pull request may close these issues.

[Bug]: Rspack dependOn not shares modules
2 participants
pFad - Phonifier reborn

Pfad - The Proxy pFad of © 2024 Garber Painting. All rights reserved.

Note: This service is not intended for secure transactions such as banking, social media, email, or purchasing. Use at your own risk. We assume no liability whatsoever for broken pages.


Alternative Proxies:

Alternative Proxy

pFad Proxy

pFad v3 Proxy

pFad v4 Proxy