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.

Sign up for GitHub

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

Jump to bottom

Argument order of transform() function (or attempt to remove done) #498

Closed
tyoshino opened this issue Aug 4, 2016 · 0 comments
Closed

Argument order of transform() function (or attempt to remove done) #498

tyoshino opened this issue Aug 4, 2016 · 0 comments
Labels
transform streams

Comments

@tyoshino
Copy link
Member

tyoshino commented Aug 4, 2016

#488 (comment)
isonmad pushed a commit to isonmad/streams that referenced this issue Sep 18, 2016
Return a Promise from TransformStream transform()
3d7eb5c 
Align with the WritableStream API.

This resolves issues whatwg#518, whatwg#498, whatwg#331, and whatwg#185.
@isonmad isonmad mentioned this issue Sep 18, 2016
Merged
isonmad pushed a commit to isonmad/streams that referenced this issue Sep 19, 2016
Return a Promise from TransformStream transform()
5b2420f 
Align with the WritableStream API.

This resolves issues whatwg#518, whatwg#498, whatwg#331, and whatwg#185.
isonmad pushed a commit to isonmad/streams that referenced this issue Sep 19, 2016
Return a Promise from TransformStream transform()
2fd0945 
Align with the WritableStream API.

This resolves issues whatwg#518, whatwg#498, whatwg#331, and whatwg#185.
tyoshino pushed a commit that referenced this issue Oct 12, 2016
@isonmad @tyoshino
Update TransformStream API & misc. fixups (#519)
a957cc5 
Move TransformStream to promises based api (#519)

Fix a bunch of bugs along the way and add tests for them.

Closes #518, #498, #331, and #185.
@domenic domenic closed this as completed Oct 12, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
transform streams
Milestone
No milestone
Development

No branches or pull requests
2 participants
@domenic @tyoshino