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

feat: optimization.chunkIds supports 'natural' #5104

Closed

Conversation

OceanPresentChao
Copy link
Contributor

@OceanPresentChao OceanPresentChao commented Dec 22, 2023

Summary

issue: issue4823

This Plugin supports 'natural' in optimization.moduleIds and optimization.chunkIds, aligning with webpack

Test Plan

run test in packages/rspack/test/optimization/natural-chunks & natural-modules

Require Documentation?

  • No
  • Yes, the corresponding rspack-website PR is __

@OceanPresentChao OceanPresentChao requested a review from a team December 22, 2023 18:20
@github-actions github-actions bot added the release: feature release: feature related release(mr only) label Dec 22, 2023
@OceanPresentChao
Copy link
Contributor Author

OceanPresentChao commented Dec 22, 2023

I keep the default value of chunkIds & moduleIds as they used to be, because the change of default value leads to changes in the output file name, failing in some test cases.
It is noticeable that the default value may not align with webpack

style: clippy

fix: revert test config

fix: revert default moduleIds
Copy link

stale bot commented Mar 24, 2024

This pull request has been automatically marked as stale because it has not had recent activity. If this pull request is still relevant, please leave any comment (for example, "bump").

@stale stale bot added the stale label Mar 24, 2024
@LingyuCoder
Copy link
Collaborator

Thanks for your contribution so much and sorry for not responding.

The optimization.chunkIds=natural and optimization.moduleIds=natural have been implemented so this pr will be closed.

Welcome your future contribution.

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) stale
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants