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
docs: add docs about defaultIgnore config #1088
Conversation
### `defaultIgnore` | ||
|
||
The default value for the [`ignore` option](queries/bytext.mdx#ignore) used by | ||
[`getByText`](queries/bytext.mdx). Defaults to `script, style`. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Now that I think about it: It's not just *ByText
.This also affects the nodes we print in error messages.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@eps1lon do you have a specific wording you'd like me to put here? 👀
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@eps1lon I gave it a shot. 😅 What do you think?
bcb15e0
to
5c90b00
Compare
@all-contributors add @robin-drexler for docs |
I've put up a pull request to add @robin-drexler! 🎉 |
Docs for testing-library/dom-testing-library#1138