-
Notifications
You must be signed in to change notification settings - Fork 941
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
[all] Drop IE11 support #3756
[all] Drop IE11 support #3756
Conversation
justinfagnani
commented
Mar 24, 2023
•
edited
Loading
edited
- Removes the IE runner and sauce configs
- Removes IE example configs in starter kits
🦋 Changeset detectedLatest commit: 48a9bed The changes in this PR will be included in the next version bump. This PR includes changesets to release 29 packages
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
📊 Tachometer Benchmark ResultsSummarynop-update
render
update
update-reflect
Resultslit-element-list
render
update
update-reflect
lit-html-kitchen-sink
render
update
nop-update
lit-html-repeat
render
update
lit-html-template-heavy
render
update
reactive-element-list
render
update
update-reflect
|
9acef39
to
48a9bed
Compare
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.
approving with comment on the packages mentioned in changeset.
'lit-html': major | ||
'lit': major | ||
'lit-element': major | ||
'@lit/reactive-element': major |
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.
theoretically the change is part of the initiative in dropping ie11 support but there were no changes to the vended package so do these need to be added to changeset? i think it only makes sense to keep the starter kits.
it will add a line in the changelog that says "Drop IE11 support" in all the mentioned packages and if that's the intention, then i guess it makes sense. though i would think we'd want to associate actual code changes to that idea. we might need to reorganize the changelog instead of relying on what's generated by changesets for this major release then.
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.
oh, i see there are PRs meant to be merged into this branch, sorry i guess i looked at these in the wrong order, though the PR title here is misleading then.
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.
Yeah, I retroactively made the the base for everything else. So "Drop IE" is accurate in that this drops support... the other PRs take advantage of that. I can change the title of the PR though.