• 
      

    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:
    Completed
    Change Summary:
    Pushed to release-4.x (9cdb5c7)