Skip to content

Conversation

@edg2s
Copy link

@edg2s edg2s commented Nov 3, 2025

No description provided.

@G-Rath G-Rath changed the title Add missing documentation of ignoreTypeOfTestName option docs(valid-title): add documentation for ignoreTypeOfTestName option Nov 3, 2025
Copy link
Collaborator

@G-Rath G-Rath left a comment

Choose a reason for hiding this comment

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

thanks, this is good as-is but if you have the time it'd be good to have this also document ignoreTypeOfDescribeName.

Happy if you just want to say no to that though, as I can do it in a follow up 🙂

When enabled, the type of the first argument to `describe` blocks won't be
checked.

#### `ignoreTypeOfTestName`
Copy link
Collaborator

Choose a reason for hiding this comment

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

if we're going to add a section for this, we might as well use it for ignoreTypeOfDescribeName too, like mustMatch and mustNotMatch below

@G-Rath G-Rath self-requested a review November 3, 2025 19:45
Copy link
Collaborator

@G-Rath G-Rath left a comment

Choose a reason for hiding this comment

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

oh actually this does need to be formatted with prettier before it can be landed 😅

@edg2s
Copy link
Author

edg2s commented Nov 4, 2025

thanks, this is good as-is but if you have the time it'd be good to have this also document ignoreTypeOfDescribeName.

Happy if you just want to say no to that though, as I can do it in a follow up 🙂

That is already documented.

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.

2 participants