Skip to content

Sync js-api tests from wpt. #1422

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
Closed
wants to merge 1 commit into from
Closed

Sync js-api tests from wpt. #1422

wants to merge 1 commit into from

Conversation

Ms2ger
Copy link
Collaborator

@Ms2ger Ms2ger commented Feb 11, 2022

No description provided.

@Ms2ger Ms2ger requested a review from gahaas February 11, 2022 09:33
gahaas
gahaas approved these changes Feb 11, 2022
View reviewed changes
Copy link
Collaborator

@gahaas gahaas left a comment

Choose a reason for hiding this comment

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

Thanks!

@rossberg
Copy link
Member

rossberg commented May 2, 2022

Is there a reason not to land this?

@Ms2ger
Copy link
Collaborator Author

Ms2ger commented May 2, 2022

I'm guessing I missed the review, and now #1449 landed everything except the v128 change, which should probably go into wpt instead.

@Ms2ger Ms2ger closed this May 2, 2022
@rossberg rossberg deleted the sync-tests branch May 2, 2022 09:09
@rossberg
Copy link
Member

rossberg commented May 2, 2022

Okay, thanks. I deleted the PR branch from the repo.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@gahaas gahaas gahaas approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@Ms2ger @rossberg @gahaas