Skip to content

v1 of accname name computation tests#39604

Merged
cookiecrook merged 9 commits intoweb-platform-tests:masterfrom
cookiecrook:accname-v1
Apr 25, 2023
Merged

v1 of accname name computation tests#39604
cookiecrook merged 9 commits intoweb-platform-tests:masterfrom
cookiecrook:accname-v1

Conversation

@cookiecrook
Copy link
Copy Markdown
Contributor

@cookiecrook
Copy link
Copy Markdown
Contributor Author

cookiecrook commented Apr 20, 2023

The log of the unsuccessful wpt-decision-task check means nothing to me, and the "re-run" button doesn't work. Any ideas are appreciated.

@cookiecrook
Copy link
Copy Markdown
Contributor Author

Okay @spectranaut and @scottaohara this is really ready for review now that the PR checks have completed successfully.

Copy link
Copy Markdown
Contributor

@spectranaut spectranaut left a comment

Choose a reason for hiding this comment

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

I would like to hear your thoughts on my one suggestion before approving!

Copy link
Copy Markdown
Contributor

@spectranaut spectranaut left a comment

Choose a reason for hiding this comment

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

thx, this is a great start!

@cookiecrook
Copy link
Copy Markdown
Contributor Author

Hmmm… auto-merge bypassed the "do not merge yet" label.

@cookiecrook
Copy link
Copy Markdown
Contributor Author

But it looks like it got everything, so I think that's okay?

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

AccName tests v1 Discuss how to break up the computedlabel tests in wpt/accname (to verify AccName computation)

6 participants