Skip to content

docs(prefer-screen-queries): update allowed properties and methods #158

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

Conversation

gndelia
Copy link
Collaborator

@gndelia gndelia commented Jun 10, 2020

Requires rebasing after #157 is merged. The changes are in this commit f17275e

edit: The properties + methods I included come from the typings here https://github.com/DefinitelyTyped/DefinitelyTyped/blob/master/types/testing-library__react/index.d.ts#L19

@gndelia gndelia force-pushed the docs/update-prefer-screen-queries-to-include-container branch from f17275e to 96dd651 Compare June 10, 2020 23:16
@Belco90
Copy link
Member

Belco90 commented Jun 11, 2020

Nice one! #157 is merged so you can rebase it now.

@gndelia gndelia force-pushed the docs/update-prefer-screen-queries-to-include-container branch from 96dd651 to 377f21d Compare June 11, 2020 15:19
{
code: `
const screen = render(baz);
screen.container.querySelector('foo');
Copy link
Member

Choose a reason for hiding this comment

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

This is a good edge case 💯

@gndelia gndelia force-pushed the docs/update-prefer-screen-queries-to-include-container branch from 377f21d to 54dd53e Compare June 11, 2020 15:27
@Belco90 Belco90 changed the title Docs: update prefer-screen-queries with allowed properties and methods docs(prefer-screen-queries): update allowed properties and methods Jun 11, 2020
@Belco90 Belco90 merged commit fd8a362 into testing-library:master Jun 11, 2020
@Belco90
Copy link
Member

Belco90 commented Jun 12, 2020

🎉 This PR is included in version 3.2.3 🎉

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
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants