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

Streams: remove obsolete test for @@asyncIterator === values #24427

Merged
merged 1 commit into from Jul 3, 2020
Merged

Streams: remove obsolete test for @@asyncIterator === values #24427

merged 1 commit into from Jul 3, 2020

Conversation

MattiasBuelens
Copy link
Contributor

As noted in #24266, this Streams test is now covered by the new async iterable tests from the WebIDL harness. Therefore, we can remove the Streams-specific test.

ricea
ricea approved these changes Jul 3, 2020
View reviewed changes
Copy link
Contributor

@ricea ricea left a comment

Choose a reason for hiding this comment

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

lgtm

@ricea ricea merged commit 67a169b into web-platform-tests:master Jul 3, 2020
@MattiasBuelens MattiasBuelens deleted the streams-remove-obsolete-test branch October 12, 2020 22:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ricea ricea ricea approved these changes

@domenic domenic Awaiting requested review from domenic

@wanderview wanderview Awaiting requested review from wanderview

@youennf youennf Awaiting requested review from youennf

@yutakahirano yutakahirano Awaiting requested review from yutakahirano

Assignees

@yutakahirano yutakahirano

Labels
streams
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
4 participants
@MattiasBuelens @ricea @yutakahirano @wpt-pr-bot