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.

Sign up for GitHub

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

Jump to bottom

Wrap classes with decorators or static properties in an IIFE, even for ES2015+ #32011

Merged
merged 2 commits into from
Feb 15, 2020
Merged

Wrap classes with decorators or static properties in an IIFE, even for ES2015+ #32011

merged 2 commits into from
Feb 15, 2020

Commits on Feb 6, 2020

  1. Always wrap classes with decorators or static properties in an IIFE 

    Currently only script targets less than or equal to ES5 will wrap classes.  However, the wrapping is also crucial to file size optimizations for ES2015+ as well.  Without the IIFE wrapper, minification tools do not elide the class.  This is due to references to the class being present within the downlevelled decorator and static property code.
This change represents the full completion of issue microsoft#15857
    @clydin
    clydin committed Feb 6, 2020
    Configuration menu
    Copy the full SHA
    6ef2923 View commit details
    Browse the repository at this point in the history

Commits on Feb 11, 2020

  1. Accept new baselines

    @clydin
    clydin committed Feb 11, 2020
    Configuration menu
    Copy the full SHA
    9ff43af View commit details
    Browse the repository at this point in the history