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
Include module file and context dependencies in ConcatenatedModule #5137
Include module file and context dependencies in ConcatenatedModule #5137
Conversation
As ModuleConcatenationPlugin removes the concatenated modules from a compilation, the file and context dependencies of those modules needs to be stored in the ConcatenatedModule for webpack to be able to watch those paths.
30c95fd
to
9ea40f0
Compare
Happy to add tests. Just need direction on that. Not sure where to add an appropriate test. |
Thank you for your pull request! The most important CI builds succeeded, we’ll review the pull request soon. |
You can add a watchTest -> |
ModuleConcatenationPlugin creates a ConcatenatedModule from harmony modules. It must be rebuilt when those modules change.
Test added. Let me know if you want any changes. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Awesome that you included a watch test 👍
Thanks |
What kind of change does this PR introduce?
bugfix #5132
Did you add tests for your changes?
Yes
If relevant, link to documentation update:
N/A
Summary
#5132
As ModuleConcatenationPlugin removes the concatenated modules from a
compilation, the file and context dependencies of those modules needs
to be stored in the ConcatenatedModule for webpack to be able to watch
those paths.
Does this PR introduce a breaking change?
No
Other information