-
Notifications
You must be signed in to change notification settings - Fork 22.5k
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
WebExtensions - fix up all the tags except for apis #25117
Conversation
@rebloor , I've assigned this to you but if you like I am happy to review it. |
Actually one thing I would say is that this PR adds a lot of So in this PR we should suppress Note that the BCD does not list these features as non-standard, which is a good thing since it means the automation that keeps |
@wbamberg seems like you might be on the ball with this one. Would you like to finish commenting and reviewing? From my POV your comment about |
Thanks @rebloor . I'm happy to review :). But actually I agree about the size of this, because GitHub hides diffs beyond a certain number of files. @hamishwillee could you split this into say 3 PRs, as well as remove |
292223b
to
d9235ac
Compare
Sure! This is now all the webext except for /api - with "non-standard" removed. There are still some "deprecated". New PRs to follow for APIs. |
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.
🎉
Fixes up all WebExtension tags. Yes, this is a huge PR. But then only one review. I scanned through them all and script still seems to be working well.
Part of mdn/mdn#262