-
-
Notifications
You must be signed in to change notification settings - Fork 84
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
Fix clippy lints #504
Fix clippy lints #504
Conversation
8ace747
to
ada37fd
Compare
Not exactly sure what to do about the remaining CI failures. |
Thanks for your work and apologies for the long wait! The argmin-math tests are broken due to #509, will have to fix that in another PR or I may just do it in this one. |
ed347fc
to
30bea6c
Compare
Fixes #509 |
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #504 +/- ##
==========================================
- Coverage 92.08% 91.49% -0.60%
==========================================
Files 178 178
Lines 24455 23623 -832
==========================================
- Hits 22520 21613 -907
- Misses 1935 2010 +75 ☔ View full report in Codecov by Sentry. |
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 for your work! :)
Replace
std::*::EPSILON
with*::EPSILON
, etc.