-
Notifications
You must be signed in to change notification settings - Fork 72
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
Use new Streams algorithm and mixins #224
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I really like leaning into IDL more heavily for doing the promise wrapping. I should do that more often.
|
||
<li><p>Set <a>this</a>'s <a for=GenericTransformStream>transform</a> to <var>transform</var>. | ||
<li><p>Set <a>this</a>'s <a for=GenericTransformStream>transform</a> to the result of | ||
<a for=TransformStream>creating</a> a {{TransformStream}} with |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
<a for=TransformStream>creating</a> a {{TransformStream}} with | |
<a for=TransformStream>creating</a> a {{TransformStream}} object with |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This mismatches the algorithm declaration, which does not include "object".
Co-authored-by: Andreu Botella <abb@randomunok.com>
5ec8d08
to
d97b9f2
Compare
I rebased this and pushed some nits and corrected things that seemed wrong. Still needs upstream changes to merge. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm
Thank you for catching those; your changes LGTM. Will work on getting upstream working now. |
The Streams PR is merged; once the link databases update let's re-run CI and see if this works. |
Follows whatwg/streams#1073.
Preview | Diff