-
Notifications
You must be signed in to change notification settings - Fork 3.5k
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
ARROW-12702: [JS] Update webpack and terser #10275
Conversation
output: { comments: false, beautify: false }, | ||
compress: { unsafe: true }, | ||
mangle: true, | ||
safari10: true // <-- works around safari10 bugs, see the "safari10" option here: https://github.com/terser-js/terser#minify-options |
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.
Safari 10 is sufficiently old enough that we don't need this anymore.
@@ -83,7 +81,7 @@ const minifyTask = ((cache, commonConfig) => memoizeTask(cache, function minifyJ | |||
}))({}, { | |||
resolve: { mainFields: [`module`, `main`] }, | |||
module: { rules: [{ test: /\.js$/, enforce: `pre`, use: [`source-map-loader`] }] }, | |||
output: { filename: '[name].js', library: mainExport, libraryTarget: `umd`, umdNamedDefine: true }, | |||
output: { filename: '[name].js', library: mainExport, libraryTarget: `umd`, umdNamedDefine: true, globalObject: 'this' }, |
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.
No idea why this is needed.
@domoritz Could you rebase on master to merge this? |
I merged master so this should be good to squash and merge. |
Thanks. I'll merge this. |
Closes apache#10275 from domoritz/webpack Authored-by: Dominik Moritz <domoritz@gmail.com> Signed-off-by: Sutou Kouhei <kou@clear-code.com>
No description provided.