-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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
Add missing spec URLs for API features #16051
Merged
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Remove trusted-types links, as these are redundant links
This pull request has merge conflicts that must be resolved before we can merge this. |
This pull request has merge conflicts that must be resolved before we can merge this. |
This pull request has merge conflicts that must be resolved before we can merge this. |
This was referenced May 11, 2022
sideshowbarker
approved these changes
Jun 1, 2022
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This PR supersedes #12709. This PR adds many new spec URLs to the API data based upon data from
@webref/idl
.After rebasing #12709, I ran the script introduced in #16052 to fix the property sorting and move the spec URLs up in the JSON, then re-ran https://gist.github.com/gsnedders/0a475683482552a47e65d55df7bd5289 to add the missing spec URLs.
Notice: if there are any changes where other properties are moved around, they were not meant to be included this PR and can be reverted. Reviewers with edit access to BCD are more than welcome to make changes directly to this PR.
I don't expect this PR to be merged in one go due to its sheer size. Rather, I'm planning to separate this into multiple PRs and use this one as a breakdown to show what changes remain.