Re: [whatwg/streams] ReadableStream.from(asyncIterable) (#1083)

I can't think of any more interesting tests to add, so I'm putting this up for review. 🙂

Quick recap of the discussions so far:
* An optional `queuingStrategy` argument is currently left as a future extension. However, [Domenic is concerned](https://github.com/whatwg/streams/pull/1083#discussion_r527221755) that implementers might not be motivated to add a convenience method if they already know that they might have to revisit it in the future.
* The high water mark of the stream is currently set to 0 (instead of the default value of 1). [The question remains](https://github.com/whatwg/streams/pull/1083#discussion_r549040797) whether we should keep it at 0 (to align with `TransformStream.readable`) or revert it to 1 (to restore the default).

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/whatwg/streams/pull/1083#issuecomment-803166084

Received on Friday, 19 March 2021 22:18:27 UTC