Skip to content

Conversation

@SebastianZ
Copy link
Contributor

I added all the Web Platform Tests that currently cover the features in Selectors 4.

For the reviewers: Please especially check whether the tests under old-tests/ are placed in the correct section and the tests in "Features from other specs and unrelated tests".

Note that this PR does not include hints about missing WPT coverage. This needs to be done separately.

(This is part of the effort to get Selectors 4 into shape for CR, see #12799 (comment).)

Sebastian

Copy link
Contributor

@svgeesus svgeesus left a comment

Choose a reason for hiding this comment

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

This looks reasonable to me on a quick diff inspection of test names (meaning I have not grepped for rel="help" in the tests, far less read the source of each test)

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

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants