Skip to content

feat: supports chunkGraph and moduleGraph for cacheGroup.test#9248

Merged
JSerFeng merged 1 commit intomainfrom
feat/chunk-graph-in-split-chunks
Feb 17, 2025
Merged

feat: supports chunkGraph and moduleGraph for cacheGroup.test#9248
JSerFeng merged 1 commit intomainfrom
feat/chunk-graph-in-split-chunks

Conversation

@JSerFeng
Copy link
Copy Markdown
Contributor

Summary

close #8988

Supports chunkGraph and moduleGraph for cacheGroup.test, and update document

Checklist

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

@JSerFeng JSerFeng requested a review from chenjiahan February 11, 2025 12:54
@github-actions github-actions bot added release: feature release: feature related release(mr only) team The issue/pr is created by the member of Rspack. labels Feb 11, 2025
@netlify
Copy link
Copy Markdown

netlify bot commented Feb 11, 2025

Deploy Preview for rspack ready!

Name Link
🔨 Latest commit 0467b9b
🔍 Latest deploy log https://app.netlify.com/sites/rspack/deploys/67b2c1ef5711ef000875de9f
😎 Deploy Preview https://deploy-preview-9248--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
Copy Markdown

codspeed-hq bot commented Feb 11, 2025

CodSpeed Performance Report

Merging #9248 will not alter performance

Comparing feat/chunk-graph-in-split-chunks (0467b9b) with main (2bfb80a)

Summary

✅ 6 untouched benchmarks

@JSerFeng JSerFeng force-pushed the feat/chunk-graph-in-split-chunks branch from 88533c6 to c5908fe Compare February 12, 2025 03:40
ahabhgk
ahabhgk previously approved these changes Feb 12, 2025
Copy link
Copy Markdown
Contributor

@ahabhgk ahabhgk left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

cc @SyMind

@SyMind
Copy link
Copy Markdown
Member

SyMind commented Feb 12, 2025

The current implementation is not good.

In a compilation lifecycle on the JS side, will create multiple moduleGraph and chunkGraph instances.

I don't think it's necessary to modify the code on the Rust side, just use the compiler._lastCompilation to get moduleGraph and chunkGraph.

@JSerFeng JSerFeng force-pushed the feat/chunk-graph-in-split-chunks branch from b8e5265 to 8c08933 Compare February 13, 2025 03:55
@JSerFeng JSerFeng requested a review from SyMind February 13, 2025 10:26
@JSerFeng JSerFeng marked this pull request as draft February 14, 2025 10:03
@JSerFeng
Copy link
Copy Markdown
Contributor Author

Wait for rebase

@JSerFeng JSerFeng force-pushed the feat/chunk-graph-in-split-chunks branch from 5b4ff4b to d73a4e2 Compare February 17, 2025 04:11
@JSerFeng JSerFeng force-pushed the feat/chunk-graph-in-split-chunks branch from d73a4e2 to 0467b9b Compare February 17, 2025 04:58
@JSerFeng JSerFeng marked this pull request as ready for review February 17, 2025 05:00
@JSerFeng JSerFeng merged commit 3e65b62 into main Feb 17, 2025
30 checks passed
@JSerFeng JSerFeng deleted the feat/chunk-graph-in-split-chunks branch February 17, 2025 06:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

release: feature release: feature 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.

[Feature]: Support ChunkGraph option in CacheGroup['test'] of SplitChunksPlugin

4 participants