-
Notifications
You must be signed in to change notification settings - Fork 80
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
When can webpack 5 be perfectly supported? #174
Comments
I'm using Webpack 5 and I don't have any issue using this plugin. What's your issue? |
|
I do need this in webpack 5 with CRA V5 ~ |
@w-t-w I'm guessing you can simply instantiate both plugins and be on your own with minimization:
Initialize TerserPlugin to your own taste https://webpack.js.org/plugins/terser-webpack-plugin/#options So that minimizer array doesn't have an '...' item and speed measure plugin would work for you. |
@g3tr1ght So I have to download terser-webpack-plugin without the convenience of webpack mode:'production', |
@w-t-w terser-webpack-plugin is a transitive dependency for you in any case, you either pin it in your package.json and instantiate yourself (make it direct dependency) or let webpack instantiate it for you (keep it transitive). In either case, this dependency will end up installed: https://github.com/webpack/webpack/blob/main/package.json#L29 Here's what Which means it actually solves the problem. I understand it's a workaround, but the end result is identical to the webpack "baseline". And more so, serious projects usually have a good number of customizations which override webpack default production optimizations. |
any updates on this? |
@ssmartin I'm also still getting this on every build, and it's driving me nuts. Everything is working and the build runs just fine, I just consistently get that warning. |
It is a great plugin.
The text was updated successfully, but these errors were encountered: