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

Test improvements #9

Merged
merged 10 commits into from
Jul 30, 2024
Merged

Test improvements #9

merged 10 commits into from
Jul 30, 2024

Conversation

antonok-edm
Copy link
Collaborator

just a couple of minor changes I made to make these easier to test manually (i.e. without Playwright)

@antonok-edm antonok-edm requested a review from pes10k July 29, 2024 02:59
@antonok-edm antonok-edm self-assigned this Jul 29, 2024
Copy link
Collaborator

@pes10k pes10k left a comment

Choose a reason for hiding this comment

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

lgtm

@antonok-edm antonok-edm force-pushed the test-improvements branch 2 times, most recently from adbaca4 to 4f60c80 Compare July 30, 2024 17:42
@antonok-edm antonok-edm force-pushed the test-improvements branch 2 times, most recently from 648df82 to 556fcc4 Compare July 30, 2024 18:36
note that "relative selectors"[0] will be canonicalized by adblock-rust
at parsing time to have an explicit leading descendant combinator rather
than an implicit one.

[0] - https://developer.mozilla.org/en-US/docs/Web/CSS/CSS_selectors/Selector_structure#relative_selector
@antonok-edm antonok-edm merged commit 0ac4794 into main Jul 30, 2024
2 checks passed
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.

2 participants