Skip to content
This repository has been archived by the owner on Jun 21, 2022. It is now read-only.

Remove Edge Mobile from compat table #1182

Merged
merged 1 commit into from
Jun 13, 2019
Merged

Remove Edge Mobile from compat table #1182

merged 1 commit into from
Jun 13, 2019

Conversation

Elchi3
Copy link
Member

@Elchi3 Elchi3 commented Jun 13, 2019

We're dropping Edge Mobile from the data and the rendered tables. See mdn/browser-compat-data#3888 (comment)

@Elchi3 Elchi3 added the bcd label Jun 13, 2019
@Elchi3 Elchi3 requested a review from wbamberg June 13, 2019 15:13
@Elchi3
Copy link
Member Author

Elchi3 commented Jun 13, 2019

@wbamberg Since you probably work with this code as part of the stumptown bcd renderer, this might be relevant to you. It would be awesome if you could review this PR, but feel free to not and then I will try to find someone else. Thanks!

Copy link

@wbamberg wbamberg left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

These changes are nice and localized and look safe :). Tests pass, and I've tested locally with a few different compat tables, including api, javascript, webextensions, and ones with just one item and ones with many. And they all look fine.

@wbamberg wbamberg merged commit 2f7ff15 into mdn:master Jun 13, 2019
@Elchi3 Elchi3 deleted the compat-rm-edge_mobile branch June 14, 2019 08:54
@Elchi3
Copy link
Member Author

Elchi3 commented Jun 14, 2019

Thank you so much, Will! ❤️

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants