Skip to content

fix: use HookMap for runtimeRequirementInTree#7987

Merged
LingyuCoder merged 4 commits intoweb-infra-dev:mainfrom
colinaaa:runtime-requirement-in-tree
Sep 26, 2024
Merged

fix: use HookMap for runtimeRequirementInTree#7987
LingyuCoder merged 4 commits intoweb-infra-dev:mainfrom
colinaaa:runtime-requirement-in-tree

Conversation

@colinaaa
Copy link
Copy Markdown
Collaborator

@colinaaa colinaaa commented Sep 25, 2024

Summary

Caution

This PR is a breaking change because compilation.hooks.runtimeRequirementInTree.tap is not avaliable

Change compilation.hooks.runtimeRequirementInTree to HookMap. This is aligned with Webpack.

Checklist

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

@netlify
Copy link
Copy Markdown

netlify bot commented Sep 25, 2024

Deploy Preview for rspack canceled.

Built without sensitive environment variables

Name Link
🔨 Latest commit fbcd425
🔍 Latest deploy log https://app.netlify.com/sites/rspack/deploys/66f4f19089fccd0009beb2b3

@colinaaa
Copy link
Copy Markdown
Collaborator Author

This PR should be merged after rstackjs/rspack-lite-tapable#5 is merged.

@github-actions github-actions bot added the release: bug fix release: bug related release(mr only) label Sep 25, 2024
This is a breaking change to Rspack 1.0.6.
@colinaaa colinaaa force-pushed the runtime-requirement-in-tree branch from e6e5ff2 to 66fc108 Compare September 26, 2024 02:37
@LingyuCoder LingyuCoder requested a review from ahabhgk September 26, 2024 02:55
@LingyuCoder LingyuCoder merged commit 5df32bd into web-infra-dev:main Sep 26, 2024
@LingyuCoder LingyuCoder changed the title fix!: use HookMap for runtimeRequirementInTree fix: use HookMap for runtimeRequirementInTree Sep 26, 2024
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)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants