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
AnyInsteadOfEmpty
In Lucky, we have a query method .any?. This actually runs SQL, so Performance/AnyInsteadOfEmpty rule here doesn't apply.
.any?
Performance/AnyInsteadOfEmpty
This code passed on 0.14.4, but fails on 1.0.0
TransactionQuery.new .receiver(user) .any? # ameba:disable Performance/AnyInsteadOfEmpty
src/serializers/post_serializer.cr:80:24 [Correctable] [C] Performance/AnyInsteadOfEmpty: Use `!{...}.empty?` instead of `{...}.any?` > TransactionQuery.new ^
Maybe solved by #265 ?
The text was updated successfully, but these errors were encountered:
That's a regression, and shouldn't depend on #265 in any way.
Sorry, something went wrong.
Sija
Successfully merging a pull request may close this issue.
In Lucky, we have a query method
.any?
. This actually runs SQL, soPerformance/AnyInsteadOfEmpty
rule here doesn't apply.This code passed on 0.14.4, but fails on 1.0.0
Maybe solved by #265 ?
The text was updated successfully, but these errors were encountered: