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
add asset modules concatenation #15515
Conversation
For maintainers only:
|
96d18b1
to
5f86eb5
Compare
1f6d4a1
to
1c0f5c7
Compare
1c0f5c7
to
3ec8a60
Compare
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.
NormalModule.codeGeneration still runs code generation for all source types, but it only has to run it for source types that remain in the chunk...
I'm also a bit worries about performance drawbacks that comes with these many accesses to sourceTypesByModule
, but let's finish that up and measure afterwards...
da03829
to
1b7bd14
Compare
- in ConcatenatedModule generate only js - in NormalModule generate only used sources
1b7bd14
to
3f1715f
Compare
f23ba1d
to
19d1a93
Compare
Thanks |
What kind of change does this PR introduce?
feature
closes #13764
Did you add tests for your changes?
yes
Does this PR introduce a breaking change?
no
What needs to be documented once your changes are merged?
nothing