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(hook): add optimizeModules hook #2758

Merged

Conversation

nieyuyao
Copy link
Contributor

Related issue (if exists)

fix #2423

@changeset-bot
Copy link

changeset-bot bot commented Apr 15, 2023

🦋 Changeset detected

Latest commit: 8c343dc

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 17 packages
Name Type
@rspack/binding Patch
@rspack/core Patch
@rspack/postcss-loader Patch
webpack-test Patch
@rspack/cli Patch
@rspack/dev-middleware Patch
@rspack/dev-server Patch
@rspack/plugin-html Patch
benchmarkcase-rspack-react-refresh Patch
@rspack/dev-client Patch
@rspack/plugin-minify Patch
@rspack/plugin-node-polyfill Patch
@rspack/binding-darwin-arm64 Patch
@rspack/binding-darwin-x64 Patch
@rspack/binding-linux-x64-gnu Patch
@rspack/binding-win32-x64-msvc Patch
@rspack/fs Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@IWANABETHATGUY
Copy link
Contributor

looks good to me, @jerrykingxyz would you mind having a look if this pr fits your needs?

@IWANABETHATGUY
Copy link
Contributor

One more thing, you should add your newly added hooks here to make the hooks filter work, https://github.com/web-infra-dev/rspack/pull/2758/files#diff-65c9f2211aa3417321f820555fd1ec4a9647fdc6cba3c80c2b96587a4d4ad4fdR330-R331

Copy link
Contributor

@IWANABETHATGUY IWANABETHATGUY left a comment

Choose a reason for hiding this comment

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

LGTM, thanks!

@IWANABETHATGUY IWANABETHATGUY added this pull request to the merge queue Apr 21, 2023
Merged via the queue into web-infra-dev:main with commit c1720f9 Apr 21, 2023
@hyf0 hyf0 changed the title feat(hook): add optimizeModules hook feat(hook): add optimizeModules hook Apr 21, 2023
@github-actions github-actions bot mentioned this pull request Apr 25, 2023
siyou pushed a commit to siyou/rspack that referenced this pull request May 14, 2023
* feat(hook): add optimizeModules hook

* fix(hook): call updateDisabledHooks method

* fix(hook): add optimizeModules to hookMap
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Feature Request]: support compilation.hooks.optimizeModules
4 participants