Skip to content
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

Performance Audit 2017: Uglify3 #177

Closed
nicjansma opened this issue Dec 22, 2017 · 2 comments
Closed

Performance Audit 2017: Uglify3 #177

nicjansma opened this issue Dec 22, 2017 · 2 comments

Comments

@nicjansma
Copy link

@nicjansma nicjansma commented Dec 22, 2017

We're currently using Uglify2 for minification.

A quick review of Uglify3 suggests it might shave some bytes:

  • Uglify2: 164,057 bytes
  • Uglify3: 160,490 bytes
@nicjansma

This comment has been minimized.

Copy link
Author

@nicjansma nicjansma commented Dec 27, 2018

This has been fixed in the Akamai fork and will be backported soon.

@nicjansma

This comment has been minimized.

Copy link
Author

@nicjansma nicjansma commented Jun 3, 2019

We changed from UglifyJS@2 to UglifyJS@3, while also enabling the ie8:true option for compatibility and compress.keep_fnames option to improve Boomerang Error reporting. After all of these changes, Boomerang is 2,074 bytes larger uncompressed, and -706/-723 bytes less gz/br.

Merged in as 65faac0

@nicjansma nicjansma closed this Jun 3, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
1 participant
You can’t perform that action at this time.