Skip to content
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 async iterable flavors #1135

Closed
wants to merge 3 commits into from
Closed

Add async iterable flavors #1135

wants to merge 3 commits into from

Conversation

MattiasBuelens
Copy link
Contributor

Add support for async iterable declarations per Web IDL, and emit them as .asynciterable.d.ts flavors. This is an alternative solution for #1134, and a first step towards microsoft/TypeScript#29867.

Since this is currently only used by a single interface (ReadableStream), I understand that there's limited use for this. That's why I'm opening this as a draft PR for now: as a proof-of-concept on what this could look like, and as a starting point to discuss if we want to proceed with this. (If we don't want this, that's fine too, no hard feelings. 🙂)

@github-actions
Copy link
Contributor

github-actions bot commented Sep 5, 2021

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.

@saschanaz
Copy link
Contributor

Hmm, I'd accept this after anyone actually implement that. For now I guess no real value is there.

@MattiasBuelens
Copy link
Contributor Author

Uhhh, yeah, I just realized there are currently 0 browser implementations of ReadableStream[Symbol.asyncIterator]. Woops. 😅

I'll try again next year or something. 😛

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants