-
Notifications
You must be signed in to change notification settings - Fork 2
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
More suggestions #95
Comments
PRs are welcome, though these would be new rules rather than suggestions - I would prefer we have specific issues rather than one generic one like this though, because what you're effectively asking for here is every rule for every matcher which will take a while 😅 I think I'm happy to review PRs though and continue maintaining with new rules, fixes, etc if interest picks up :) |
Sorry I mean autofix suggestions. :) We are thinking about to write them by ourself, I'll get some time for that. |
Some overlap with my report in #219 |
It'd be nice to have more suggestions:
expect(typeof x === 'string') -> expect(x).toBeString()
expect(typeof x === 'boolean') -> expect(x).toBeBoolean()
expect(typeof x === 'number') -> expect(x).toBeNumber()
expect(x.includes(y)) -> expect(x).toInclude(y)
And so on.
The text was updated successfully, but these errors were encountered: