-
-
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 TARGET_IE11 flag #7739
Remove TARGET_IE11 flag #7739
Conversation
@@ -2,12 +2,6 @@ | |||
|
|||
const browsers = ['last 1 Chrome versions', 'last 1 Firefox versions', 'last 1 Safari versions']; |
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.
Perhaps we should change this to last 2
across all? Thoughts @igorT ?
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.
We should do whatever Ember does
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.
browsers = ['last 2 Chrome versions', 'last 2 Firefox versions', 'Safari 12', 'last 2 Edge versions'];
Good idea. Here it is and updated in this PR.
Asset Size Report for 6496945 Modern Builds ✅ EmberData shrank by -39.0 B (-38.0 B compressed)WarningsChangeset
Full Asset Analysis (Modern)
Modern Builds (No Rollup) ✅ EmberData shrank by -39.0 B (-55.0 B compressed)WarningsChangeset
Full Asset Analysis (Modern)
|
2592588
to
98c2bbb
Compare
Performance Report for 6496945 Scenario - materialization: ☑️ Performance is stable
Scenario - unload: ☑️ Performance is stable
Scenario - destroy: ☑️ Performance is stable
Scenario - add-children: ☑️ Performance is stable
Scenario - unused-relationships: ☑️ Performance is stable
|
9e2b604
to
cbcf812
Compare
cbcf812
to
1cf600b
Compare
@snewcomer looks like lint is failing |
941142b
to
6496945
Compare
* Remove TARGET_IE11 flag * SHOULD_TRANSPILE just like emberjs * just do all browsers in prod env * update moar targets
* Remove TARGET_IE11 flag * SHOULD_TRANSPILE just like emberjs * just do all browsers in prod env * update moar targets
* Remove TARGET_IE11 flag * SHOULD_TRANSPILE just like emberjs * just do all browsers in prod env * update moar targets
prep for 4.0!
ref #7724
There are some comments about IE11 in the codebase and features we couldn't use due to lack of APIs. Will follow up after to remove all comments and apply necessary changes.
This PR ultimately aligns our browser targets with ember.js