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

fix(streams): reject string in ReadableStream.from type #25116

Conversation

petamoriken
Copy link
Contributor

WebIDL async iterable<T> type rejects string

Ref whatwg/webidl#1397, #24623

@crowlKats crowlKats requested a review from lucacasonato August 20, 2024 19:26
@petamoriken petamoriken changed the title fix(streams): reject string in ReadableStream.from fix(streams): reject string in ReadableStream.from type Aug 20, 2024
@petamoriken
Copy link
Contributor Author

#25207

@petamoriken petamoriken reopened this Nov 15, 2024
@petamoriken petamoriken force-pushed the fix/reject-string-in-readable-stream-from branch from e4b4ac9 to 5867a8c Compare November 15, 2024 20:20
@petamoriken
Copy link
Contributor Author

relanded: #26882

Comment on lines -63 to +64
interface ReadableStreamGenericReader<T> {
readonly closed: Promise<void>;
interface ReadableStreamGenericReader {
readonly closed: Promise<undefined>;
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I fixed differences with lib.dom.d.ts

Copy link
Member

@lucacasonato lucacasonato left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@bartlomieju bartlomieju enabled auto-merge (squash) November 26, 2024 14:15
@bartlomieju bartlomieju merged commit 4330ef5 into denoland:main Nov 26, 2024
17 checks passed
@petamoriken petamoriken deleted the fix/reject-string-in-readable-stream-from branch November 27, 2024 08:45
bartlomieju pushed a commit that referenced this pull request Nov 28, 2024
WebIDL `async iterable<T>` type rejects `string`

Ref whatwg/webidl#1397, #24623
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.

3 participants