-
-
Notifications
You must be signed in to change notification settings - Fork 140
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
chore: run clippy against all targets #546
chore: run clippy against all targets #546
Conversation
f818034
to
77c0539
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.
Good idea! Would be good to avoid the unrelated formatting changes in the GitHub Actions file though.
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.
overall, this looks good, thanks for the lint fixes! i agree with @hds that we should undo the YAML formatting changes before merging.
670d55a
to
4c6daa4
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.
🔢 Self-check (PR reviewed by myself and ready for feedback.)
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.
looks good to me, thank you!
This would help us to check if there is any clippy warning in the tests.
7f91d54
to
8cb8202
Compare
@hds This PR needs your approval before I can merge it. Could you please take a look? Thanks! |
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 adding this, great idea!
Sorry! I didn't realize I was blocking this one. 🤦♀️ |
No worry! Thank you ❤️ |
This would help us to check if there is any clippy warning in the tests.
I found it after #544. There are some unused imports in our test code.
Use
cargo clippy --workspace --all-targets --no-deps -- -D warnings
to test all targes which include the test targets.