Can we use uglify-es instead of uglify-js?

ForbesLindesay / browserify-middleware

express middleware for browserify, done right

http://browserify.org

MIT License

381 stars 66 forks source link

Can we use uglify-es instead of uglify-js? #117

Closed pigeonman99 closed 6 years ago

pigeonman99 commented 6 years ago

My javascript uses some ES6 features such as using "let" to declare a variable instead of "var". When in production mode, browserify-middleware works but was unable to minify/uglify my code. I suspect this is because browserify-middleware uses uglify-js which only support ES5.

Can we switch to uglify-es or give us an option to choose between uglify-js/uglify-es?

ForbesLindesay commented 6 years ago

I'd be happy to accept a pull request to upgrade to uglify-es

pigeonman99 commented 6 years ago

pull request created: https://github.com/ForbesLindesay/browserify-middleware/pull/118 thanks!