-
-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
Remove IE11 hacks #7741
Remove IE11 hacks #7741
Conversation
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.
These changes are good but the build tooling will have to be updated to not include IE11 as well, that's the reason for most the infra test failures.
Primarily this means updating our actions configs.
Since the IE11 size is no longer relevant we can drop that bit from the asset size report entirely.
d927121
to
f02e0f4
Compare
Asset Size Report for 24cbdd8 Modern Builds EmberData increased by +1.26 KB uncompressed but decreased by -164.0 B compressedWarnings
Changeset
Full Asset Analysis (Modern)
Modern Builds (No Rollup) EmberData increased by +1.16 KB uncompressed but decreased by -71.0 B compressedWarnings
Changeset
Full Asset Analysis (Modern)
|
@@ -22,7 +21,7 @@ module.exports = function (environment, app, compatVersion) { | |||
plugins.push([FilterImports, { imports: filteredImports }]); | |||
} | |||
|
|||
plugins.push([TransformBlockScoping, { throwIfClosureRequired: true }], ...DebugMacros); | |||
plugins.push(...DebugMacros); |
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.
@runspired This is the right removal, right?
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.
No, though it's likely also safe to remove this now. There's several GitHub actions/test run scenario infra bits that expect an IE11 compat build.
Performance Report for 24cbdd8 Scenario - materialization: ☑️ Performance is stable
Scenario - unload:
|
* Remove IE11 hacks * rm plugin-transform-block-scoping
prep for 4.0!
ref #7724