-
Notifications
You must be signed in to change notification settings - Fork 469
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
ci/Fix cargo clippy
action
#942
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #942 +/- ##
==========================================
- Coverage 87.50% 87.42% -0.09%
==========================================
Files 502 502
Lines 51080 50967 -113
==========================================
- Hits 44697 44557 -140
- Misses 6383 6410 +27 ☔ View full report in Codecov by Sentry. |
92f8601
to
86ea54f
Compare
72c2f9a
to
a073ebf
Compare
This PR is ready to be reviewd |
I rebased and CI is still failing. |
Yep, there are more clippy warnings to be fixed in the code |
82cc96f
to
9e992d6
Compare
@antimora Rebased, but there are |
ac0b6e9
to
5e26da8
Compare
Every |
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.
Thanks 🙏
Pull Request Template
Checklist
run-checks
script has been executed.Related Issues/PRs
Provide links to relevant issues and dependent PRs.
Changes
This PR fixes the
clippy
action removing the filtering based on diff that does not always allow to detect clippy warningsTesting
Describe how these changes have been tested.