-
-
Notifications
You must be signed in to change notification settings - Fork 33
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!: drop support for Internet Explorer #109
Conversation
9bbf6e9
to
22dc601
Compare
Pull Request Test Coverage Report for Build 11756318444Details
💛 - Coveralls |
Signed-off-by: Jon Koops <[email protected]> BREAKING CHANGE: drops support for Internet Explorer
22dc601
to
dc52c92
Compare
@rolandjitsu this one is ready to go, can I ask you for a review? |
There's apparently an IE mode in Edge that's supported until 2029 (see internet explorer 11 has retired)
Not sure what that translates to. There may also be clients out there that are still using IE 🤔 I just thought that's worth mentioning it. |
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.
It look ok. I just wonder if we should just drop support for IE11 🤔
We're already shipping syntax that is no longer supported in Internet Explorer, so this realistically won't be an issue. Also the compatibility mode in Edge is intended to be used only on an allow-list basis for sites that experience compatibility issues, not as a blanket feature to enable on all sites, as that would most certainly cause most sites not to function. |
@rolandjitsu any objection against merging this one? I have a couple of PRs that will build off this to resolve the remaining linting issues. |
🎉 This PR is included in version 3.0.0-beta.3 🎉 The release is available on: Your semantic-release bot 📦🚀 |
What kind of change does this PR introduce?
Did you add tests for your changes?
If relevant, did you update the documentation?
Summary
Removes conditional code and tests for Internet Explorer and drops support for it.
Does this PR introduce a breaking change?
Yes, it removes support for Internet Explorer entirely.
Other information
Not relevant.