Skip to content

script: Add error message for invalid selectors. - #47127

Merged
Narfinger merged 2 commits into
mainfrom
jdm-patch-58
Aug 10, 2026
Merged

Narfinger merged 2 commits into
mainfrom
jdm-patch-58

Conversation

@jdm

@jdm jdm commented Aug 10, 2026

Copy link
Copy Markdown
Member

This provides more useful error messages when websites are doing invalid things with selectors.

Testing: No tests for the contents of error messages.
Part of #40756

jdm added 2 commits August 10, 2026 10:39
Signed-off-by: Josh Matthews <josh@joshmatthews.net>
Signed-off-by: Josh Matthews <josh@joshmatthews.net>
@jdm
jdm requested a review from gterzian as a code owner August 10, 2026 14:41
@servo-highfive servo-highfive added the S-awaiting-review There is new code that needs to be reviewed. label Aug 10, 2026
@servo-highfive servo-highfive removed the S-awaiting-review There is new code that needs to be reviewed. label Aug 10, 2026
@Narfinger
Narfinger enabled auto-merge August 10, 2026 14:55
@Narfinger
Narfinger added this pull request to the merge queue Aug 10, 2026
@servo-highfive servo-highfive added the S-awaiting-merge The PR is in the process of compiling and running tests on the automated CI. label Aug 10, 2026
Merged via the queue into main with commit 4c54c13 Aug 10, 2026
36 checks passed
@Narfinger
Narfinger deleted the jdm-patch-58 branch August 10, 2026 16:06
@servo-highfive servo-highfive removed the S-awaiting-merge The PR is in the process of compiling and running tests on the automated CI. label Aug 10, 2026
RichardTjokroutomo pushed a commit to RichardTjokroutomo/servo that referenced this pull request Oct 6, 2026
This provides more useful error messages when websites are doing invalid
things with selectors.

Testing: No tests for the contents of error messages.
Part of servo#40756

---------

Signed-off-by: Josh Matthews <josh@joshmatthews.net>
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.

3 participants