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 WritableStream.close() #827

Closed
wants to merge 1 commit into from
Closed

Add WritableStream.close() #827

wants to merge 1 commit into from

Conversation

MattiasBuelens
Copy link
Contributor

The streams standard has added a new shorthand method to close a writable stream: WritableStream.close(). It's already implemented in Chrome 81.

@saschanaz
Copy link
Contributor

WritableStream lost an implementation so Chromium has the only implementation now, which makes it inappropriate to be added to lib.d.ts....

@MattiasBuelens
Copy link
Contributor Author

Oh right, EdgeHTML is no more, and Gecko and WebKit only support ReadableStream... 😞

I'll close this for now. If more browsers add support for the other stream APIs in the future, I can always open a new PR, no big deal. 🙂

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