-
Notifications
You must be signed in to change notification settings - Fork 439
Update core dependencies #1830
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
Update core dependencies #1830
Conversation
cd9de33
to
52c6ecc
Compare
Thanks for the PR! This section of the codebase is owned by @saschanaz - if they write a comment saying "LGTM" then it will be merged. |
* | ||
* [MDN Reference](https://developer.mozilla.org/docs/Web/API/ImageDecoder) | ||
*/ | ||
interface ImageDecoder { |
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.
This is blink only right now...
https://developer.mozilla.org/en-US/docs/Web/API/ImageDecoder#browser_compatibility
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.
MDN takes a few days after BCD release happens. Best to check BCD directly
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.
Same in the data.
133 will be released 2024-11-26 as shown nicely in mdn :)
https://github.com/mdn/browser-compat-data/blob/main/api/ImageDecoder.json
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.
That's fine, any features riding release train as enabled-by-default are treated as a support. Because:
- Being enabled by default usually means it won't go away, meaning no back and forth in the lib.
- It takes some times until it reaches to a new TS release, and by that time the browsers release will happen already.
52c6ecc
to
5312ad1
Compare
LGTM |
Merging because @saschanaz is a code-owner of all the changes - thanks! |
Automated changes by create-pull-request GitHub action