Speed up build with improvements to the minification configuration #973
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Seems like a good idea to update to the latest before the next commit, in which I'm going to try speeding up the build by switching to Terser.
Drops the build from 38s to 36s.
Two things going on here:
1. Turning off things with small gains based on this list:
https://github.com/vuejs/vue-cli/blob/4ebf6700a403ca3c8e39387684acff6fca3632ac/packages/%40vue/cli-service/lib/config/terserOptions.js
2. Switching from plug-in style to built-in `optimization` → `minimizer`
style; surprisingly, this one seemed to have the biggest impact.
With these, we go from 38s to 27s, and the build size doesn't change
prohibitively:
Before:
874084 alloy-editor-all-min.js
273571 alloy-editor-core-min.js
383670 alloy-editor-no-ckeditor-min.js
764321 alloy-editor-no-react-min.js
After:
881701 alloy-editor-all-min.js
278273 alloy-editor-core-min.js
389525 alloy-editor-no-ckeditor-min.js
770769 alloy-editor-no-react-min.js
Post-gzip, those size changes boil down to effectively nothing.
With this, first build is 27s, second build is 15s.
As documented here: https://github.com/webpack-contrib/terser-webpack-plugin This doesn't seem to make things appreciably faster though; maybe a second at most, but still, I'll take it.
|
Just started reviewing :) |
Sent from
|
@wincent great! |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
In summary: