-
-
Notifications
You must be signed in to change notification settings - Fork 3
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 #214
Comments
Assign me please! |
Thanks! Oh by the way, re:derivative, there's mcarton/rust-derivative#58. I don't mind if you choose to:
Oh and github shows clippy warnings inline, scroll down a bit on https://github.com/azriel91/autexousious/pull/215/files
|
The question might be completely dumb. But how should i run clippy in order to have the same output ? |
Think this should be it: cargo +stable clippy --all Running it twice in a row might not get you the warnings the second time (clippy needs the files to be recompiled), but if you change a file then running it again should give you the warnings ✌️. |
Much appreciated! |
Well, it seems like it's done. Should i fork a repo in order to create a PR ? |
Let's go with forking ✌️ |
Closed in #216 🚀. |
Clippy warns on
#[derivative(Debug)]
for thematch_single_binding
lint (see mcarton/rust-derivative#58), and a number of other cases.We should fix these, and at the end of the fix, the
clippy
job should be able to pass with-D warnings
.Full list of clippy warnings / output (wall of text)
The text was updated successfully, but these errors were encountered: