We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
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
this pattern is common
var arr []string assert.True(t, 0 < len(arr))
https://github.com/search?q=language%3Ago+%22assert.True%28t%2C+0+%3C+len%28%22&type=code
compares checker will fix it to
compares
var arr []string assert.Positive(t, len(l))
But that's it, while we should convert it to Empty
Empty
assert.NotEmpty(t, l)
It's a new use case
Originally posted by @ccoVeille in #119 (comment)
The text was updated successfully, but these errors were encountered:
Please confirm if the logic is valid for you, then I can code it
Sorry, something went wrong.
Well this one could be added to #119 maybe, not sure if we have to do it at the same time or separately
But this one won't be detected
Do you check it before spam issue?
Duplicate of #122 Implemented in #124
I did check locally, but apparently I failed to find it.
I checked twice, once with testifylint binary, but I didn't build it recently 🤦
Sorry
No branches or pull requests
this pattern is common
https://github.com/search?q=language%3Ago+%22assert.True%28t%2C+0+%3C+len%28%22&type=code
compares
checker will fix it toBut that's it, while we should convert it to
Empty
It's a new use case
Originally posted by @ccoVeille in #119 (comment)
The text was updated successfully, but these errors were encountered: