-
Notifications
You must be signed in to change notification settings - Fork 44
"mangle.exclude" doesn't seem to exclude the namespaces #77
Comments
@kettanaito Thanks for issue, you just use invalid configuration - use |
@evilebottnawi thanks for the reply! TL;DRPlease show me a missing example of how to provide these options using BabelMinifyPlugin. Thanks. Long versionTo be honest, I would like to see at least one working example. Even what you've posted right now doesn't make much sense to me: Here's how to use the plugin from its README: new MinifyPlugin(minifyOpts, pluginOpts) How I'm using it: new MinifyPlugin({
removeConsole: true,
mangle: { ... }
}) You suggest I should use
Please, could you clarify this? It feels like there's no clear explanation how to pass those options. P.S. I would be only glad to add the proper example of usage myself as a pull request, only explain it to me please. |
@kettanaito sorry, you are right, you have valid config, i reopen issue, but seems it is not related to plugin, we just pass option to |
@evilebottnawi thanks! Let's address the respective repository then, it seems things are broken. Do you mind doing so? If not, please just direct me what is a proper place to ask. Regardless, what do you think about adding at least one real-world example to this repo? That would help a lot, from my experience. |
@kettanaito agree, PR welcome, many work right now |
Hi. I'm using the latest version of this package -
0.2.0
.Issue
Providing
mangle.exclude
doesn't exclude the given class/method names from being mangled.Minify options
Environment
Reproduce
dev/bundle-size
branch if not there already.npm run build
to build the project in production modePlease help me resolve this issue. Thanks.
The text was updated successfully, but these errors were encountered: