-
Notifications
You must be signed in to change notification settings - Fork 70
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
chore: Update generate-rollup-config to drop older browser support #169
Conversation
"shx": "^0.3.2", | ||
"sinon": "^6.1.5", | ||
"videojs-generate-karma-config": "^5.1.0", | ||
"videojs-generate-rollup-config": "^3.1.0", | ||
"videojs-generate-karma-config": "^8.0.1", |
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.
Also update version of this package which has removed older browserstack launchers
@@ -219,7 +219,7 @@ QUnit.test('error in webkitSetMediaKeys rejects promise', function(assert) { | |||
} | |||
}; | |||
|
|||
window.WebKitMediaKeys = () => {}; | |||
window.WebKitMediaKeys = function() {}; |
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.
Arrow functions aren't constructable since they are no longer transpiled into ES5. That was throwing some errors in tests.
@@ -64,21 +64,17 @@ | |||
"npm-merge-driver-install": "^1.0.0", | |||
"npm-run-all": "^4.1.3", | |||
"pkg-ok": "^2.2.0", | |||
"rollup": "^0.65.0", | |||
"rollup": "^2.2.0", |
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.
big jump, heh
Description
This updates the default browserslist used by babel. It drops support for some older browsers including IE 11. This also removes the
browserslist
inpackage.json
so we use the default list from@videojs/babel-config
(linked above).The plan is to merge this into
main
and do a major version release.NOTE: Merge with
BREAKING CHANGE
header