-
-
Notifications
You must be signed in to change notification settings - Fork 255
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 all clippy warnings and errors #555
Conversation
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.
Everything looks good except for a double check on the application of De Morgan's here
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.
One important nit to be addressed (and maybe that address is just "well that's not necessary anymore"), then this looks good to merge
f17da19
to
3ff64a9
Compare
3ff64a9
to
8fc8bdb
Compare
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.
Sorry for the huge delay!
No worries, life has gotten very busy for me too. |
@NoahTheDuke I saw this follow up PR: #559 -- will you try to re-open it? |
I replied to you in that PR to keep things organized. |
Didn't touch any of theallow
clippy directives, but maybe I should take a look at those too.Removed unnecessary clippy directives too.