Bump mdn BCD to 8.0.5 - #2557
Adam Naji (Bashamega) wants to merge 3 commits into
Conversation
|
Thanks for the PR! This section of the codebase is owned by Kagami Sascha Rosylight (@saschanaz) - if they write a comment saying "LGTM" then it will be merged. |
| * | ||
| * [MDN Reference](https://developer.mozilla.org/docs/Web/API/ReadableStream/from_static) | ||
| */ | ||
| from(asyncIterable: never): ReadableStream; |
There was a problem hiding this comment.
Now we need to deal with this
There was a problem hiding this comment.
I just made it never what it should be?
There was a problem hiding this comment.
Something not never, likely AsyncIterable | Iterable per https://webidl.spec.whatwg.org/#js-async-iterable. And skip AsyncIterable where it's not available.
There was a problem hiding this comment.
Updated
|
Please tell the version in the PR title |
Signed-off-by: Bashamega <adambashaahmednaji@gmail.com>
| JSON.stringify(obj), | ||
| ); | ||
| return "never"; // Use 'never' to signal omission in type generation | ||
| if (compilerBehavior.treatAsyncSequence) { |
There was a problem hiding this comment.
It's not letting asynciterable.d.ts emit the extra signature, it should.
There was a problem hiding this comment.
How do we emit to asynciterable.d.ts, I tried to understand how it emits but couldn't solve it.. feel free to cherry pick the pr if you want
2 participants
No description provided.