Enable mangling in Uglify.js by default.

Review Request #12800 — Created Jan. 23, 2023 and submitted

Information

Djblets
release-4.x

Reviewers

Modern versions of Uglify don't mangle (alter code to shorten variable
names and logic) by default, which can mean losing out on some nice
reductions in file sizes. For our stuff, we want to default to mangling.
While this make a small difference for Djblets (about a 90KB savings),
it can make a much larger difference for consumers.

The arguments now include --compress --mangle, just to be thorough.
This is set automatically in build_pipeline_settings(), and can be
turned off if required.

Successfully built static media for Djblets and Review Board and tested
it across many areas of the sites.

Summary ID
Enable mangling in Uglify.js by default.
Modern versions of Uglify don't mangle (alter code to shorten variable names and logic) by default, which can mean losing out on some nice reductions in file sizes. For our stuff, we want to default to mangling. While this make a small difference for Djblets (about a 90KB savings), it can make a much larger difference for consumers. The arguments now include `--compress --mangle`, just to be thorough. This is set automatically in `build_pipeline_settings()`, and can be turned off if required.
2538416977b9c5179c2280a15abe3f61cc762947
david
  1. Ship It!
  2. 
      
chipx86
Review request changed

Status: Closed (submitted)

Change Summary:

Pushed to release-4.x (9cdb5c7)
Loading...