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

feat: [#1683] Adds support for subsequent sibling combinator to query selectors and match #1686

Conversation

karpiuMG
Copy link
Contributor

…uerySelector

@karpiuMG karpiuMG requested a review from capricorn86 as a code owner January 12, 2025 19:17
@karpiuMG karpiuMG force-pushed the 1683-getbyrole-subsequent-sibling-combinator branch 2 times, most recently from 37036db to 72a485b Compare January 12, 2025 21:16
@karpiuMG
Copy link
Contributor Author

karpiuMG commented Jan 12, 2025

@capricorn86 can you help me with the failing build (18)?

rerun helped

@karpiuMG karpiuMG force-pushed the 1683-getbyrole-subsequent-sibling-combinator branch from 72a485b to 123a765 Compare January 13, 2025 09:10
@capricorn86
Copy link
Owner

capricorn86 commented Jan 14, 2025

@capricorn86 can you help me with the failing build (18)?

rerun helped

Yeah, there are some tests that are timer based and are bit flaky that needs to be fixed. Sometimes it has to do with the fetch response to github.com and npmjs.com for the integration tests.

@capricorn86 capricorn86 changed the title fix: [#1683] Fixes not recognizing subsequent sibling combinator in Q… feat: [#1683] Adds support for subsequent sibling combinator to query selectors and match Jan 14, 2025
@capricorn86 capricorn86 merged commit 3bb23c2 into capricorn86:master Jan 14, 2025
3 checks passed
@capricorn86
Copy link
Owner

Thank you for your contribution @karpiuMG! ⭐

I made a commit to improve the performance of the loop a bit, added support for matches() and fixes to make sure that the cache is cleared if any element is changed.

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