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

Run all tests under html/dom #3155

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

ExE-Boss
Copy link
Contributor

This also migrates html/dom/historical‑dont‑upstream.html to use the new format that was introduced in #3139.

@ExE-Boss ExE-Boss force-pushed the test/web-platform-tests/run-all-html-dom-tests branch from feaf121 to 6ff2f62 Compare March 23, 2021 23:08
@domenic
Copy link
Member

domenic commented Mar 27, 2021

Can you separate these changes? Getting rid of dont-upstream.html tests is valuable, but I want to evaluate that separately from whether enabling more tests in html/dom is a good idea, and especially whether we want to enable partially-failing tests since those create a higher maintenance burden.

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.

None yet

2 participants