Skip to content
This repository has been archived by the owner on Feb 7, 2023. It is now read-only.

Updated types for 2021-12-21 #172

Closed
wants to merge 1 commit into from
Closed

Conversation

ghost
Copy link

@ghost ghost commented Dec 21, 2021

This PR includes the auto-generated types generated on 2021-12-21.

@@ -788,6 +788,7 @@ declare class ReadableStream {
pipeThrough(transform: ReadableStreamTransform, options?: PipeToOptions): ReadableStream;
pipeTo(destination: WritableStream, options?: PipeToOptions): Promise<void>;
tee(): [ReadableStream, ReadableStream];
values(options?: ReadableStreamValuesOptions): ReadableStreamReadableStreamAsyncIterator;
Copy link
Contributor

Choose a reason for hiding this comment

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

Is [Symbol.asyncIterator] implemented and just not showing up, or is that coming soon? cc @jasnell

Copy link
Member

Choose a reason for hiding this comment

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

It's landed. I think it might be going out this week.

declare class ReadableStreamReadableStreamAsyncIterator {
constructor(underlyingSource?: Object, queuingStrategy?: Object);
next(): Promise<ReadableStreamReadResult<any>>;
return(reason?: any): Promise<ReadableStreamReadResult<any>>;
Copy link
Contributor

Choose a reason for hiding this comment

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

Would be good to make all stream types generic at some point.

@JacobMGEvans
Copy link
Contributor

@threepointone Are you wanting the CI to do something like this for the Release workflow?

@threepointone
Copy link
Contributor

Closing this since we landed #181

@mrbbot mrbbot deleted the update-types-2021-12-21 branch August 30, 2022 20:41
@mrbbot mrbbot restored the update-types-2021-12-21 branch August 30, 2022 20:41
@mrbbot mrbbot deleted the update-types-2021-12-21 branch August 30, 2022 20:41
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants