Skip to content

fix: sort group by group index#9394

Merged
LingyuCoder merged 1 commit intomainfrom
fix/css-group-sort-by-index
Feb 20, 2025
Merged

fix: sort group by group index#9394
LingyuCoder merged 1 commit intomainfrom
fix/css-group-sort-by-index

Conversation

@JSerFeng
Copy link
Contributor

@JSerFeng JSerFeng commented Feb 19, 2025

Summary

sort chunk groups by its index, the index is created according to its visit order.

Align to webpack chunk.groupsIterator

Checklist

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

@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 Feb 19, 2025
@netlify
Copy link

netlify bot commented Feb 19, 2025

Deploy Preview for rspack canceled.

Name Link
🔨 Latest commit 50d0031
🔍 Latest deploy log https://app.netlify.com/sites/rspack/deploys/67b5e26a07e9af0008028d2a

@codspeed-hq
Copy link

codspeed-hq bot commented Feb 19, 2025

CodSpeed Performance Report

Merging #9394 will not alter performance

Comparing fix/css-group-sort-by-index (50d0031) with main (39f13d9)

Summary

✅ 7 untouched benchmarks

@JSerFeng JSerFeng marked this pull request as ready for review February 20, 2025 05:47
@LingyuCoder LingyuCoder merged commit f7d50be into main Feb 20, 2025
69 checks passed
@LingyuCoder LingyuCoder deleted the fix/css-group-sort-by-index branch February 20, 2025 05:48
@h-a-n-a h-a-n-a mentioned this pull request Feb 25, 2025
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.

2 participants