-
Notifications
You must be signed in to change notification settings - Fork 5.2k
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
Work around terser bug. #11758
Work around terser bug. #11758
Conversation
Hi @brianlukoff thank you for this investigation. The upgrade in the version is required to use this option or what was the reason for this bump? |
Just figured it was worthwhile upgrading while we were at it, but I don't think it is strictly necessary as |
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.
I've read the 5.9.0 change log and it seems it only contains bug fixes.
I'm waiting all the tests to complete. Also, I reinstalled AppVeyor integration as it is not communicating properly with GitHub but the tests are running here. |
AppVeyor integration is back 🎉 |
Thank you @brianlukoff. This fix is published in minifier-js@2.7.2. |
Thanks @filipenevola! |
@@ -22,6 +22,7 @@ export const meteorJsMinify = function (source) { | |||
drop_debugger: false, // remove debugger; statements | |||
unused: false, // drop unreferenced functions and variables | |||
dead_code: true, // remove unreachable code | |||
evaluate: false, // work around issue in terser (possibly https://github.com/terser/terser/issues/837) |
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.
This has caused #11775
This is intended to fix #11756, by disabling the evaluation option in terser's compressor.