-
-
Notifications
You must be signed in to change notification settings - Fork 6.3k
Add a top-level terser
option to allow users to customize the minifier
#6752
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
Conversation
Its test error reporting is confusing, and from the code logic, the test file is not written correctly |
If you have time, take a look at this pr and give some feedback Please forgive me, I'll be implementing more good solutions into Vue CLI5 in the near future🎸, so I'll probably keep bothering you And there are some new ideas I would like to get feedback from your side 🎨🏄♂️ |
Vue CLI automatically runs the |
PR updated This test has timed out, You can try re-trigger it
All tests in the local environment have passed |
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.
Looks good to me. One more thing: we should allow minify
to be a function. That could be done in a separate PR.
ok, got it |
terser
option to allow users to customize the minifier
Thanks! I'm going to merge this PR. One more thing: would you please open a new PR to update the documentation? https://github.com/vuejs/vue-cli/edit/next/docs/config/README.md |
ok, Okay, I'll finish it. |
What kind of change does this PR introduce? (check at least one)
Does this PR introduce a breaking change? (check one)
Other information
background
[email protected]
we can custom minifyesbuild
,swc
oruglifyJs
👇Some data
Vue CLI 5 demo