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

fix(prefer-presence-queries): ignore getBy* inside within for on absence queries #740

Conversation

nathanmmiller
Copy link
Contributor

…ence queries

Checks

Changes

Context

Fixes #518 [Well, partially fixes it.]

@nathanmmiller
Copy link
Contributor Author

Hey @Belco90 - thoughts on this PR? Would love to get it in to clean up this issue.

@Belco90
Copy link
Member

Belco90 commented Apr 8, 2023

@nathanmmiller Sorry, I'm not available yet. I hope I can check this by the end of the month.

@Belco90
Copy link
Member

Belco90 commented Apr 26, 2023

@nathanmmiller thanks for your patience! I'm slowly catching up, so I hope I can review it soon.

Copy link
Member

@Belco90 Belco90 left a comment

Choose a reason for hiding this comment

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

LGTM! Thanks for your contribution 💪

Copy link
Member

@Belco90 Belco90 left a comment

Choose a reason for hiding this comment

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

Oh sorry, one change: can you add some examples with the within usage to the docs?

@Belco90
Copy link
Member

Belco90 commented Aug 1, 2023

Hey @nathanmmiller! It's been a while since you created this PR. Are you still willing to contribute?

@nathanmmiller
Copy link
Contributor Author

nathanmmiller commented Aug 1, 2023 via email

@Belco90
Copy link
Member

Belco90 commented Aug 1, 2023

@nathanmmiller yes, only doc changes and it's ready to go.

@nathanmmiller
Copy link
Contributor Author

@nathanmmiller yes, only doc changes and it's ready to go.

Docs added!

@Belco90 Belco90 changed the title fix: ignore getBy inside of within for prefer-presence-queries on abs… fix(prefer-presence-queries): ignore getBy* inside within for on absence queries Aug 5, 2023
Copy link
Member

@Belco90 Belco90 left a comment

Choose a reason for hiding this comment

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

All good now! Thanks for your contribution 👌

@Belco90 Belco90 merged commit b75dc73 into testing-library:main Aug 5, 2023
26 checks passed
@Belco90
Copy link
Member

Belco90 commented Aug 5, 2023

@all-contributors please add @nathanmmiller for code and test

@allcontributors
Copy link
Contributor

@Belco90

I've put up a pull request to add @nathanmmiller! 🎉

@github-actions
Copy link

github-actions bot commented Aug 5, 2023

🎉 This PR is included in version 5.11.1 🎉

The release is available on:

Your semantic-release bot 📦🚀

@github-actions
Copy link

github-actions bot commented Aug 5, 2023

🎉 This PR is included in version 6.0.0-alpha.15 🎉

The release is available on:

Your semantic-release bot 📦🚀

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

Successfully merging this pull request may close these issues.

False positives with testing-library/prefer-presence-queries
2 participants