You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
The contribution guide suggests running clippy. There are several pull requests that use the cargo clippy fix. But at the same time, I did not find a ci job that would check clippy, which looks strange.
The text was updated successfully, but these errors were encountered:
Running clippy in CI for every PR would be very good idea to keep codebase in better shape. However, there are quite a lot of issues in the main branch currently. Majority of them should be fixed first.
However, I am interested in working on this issue.
I've been avoiding this as much as possible because I hate some things from clippy, but overall it is a good direction. I would suggest using this command as the entrypoint cargo clippy -- -A clippy::all -W clippy::correctness -W clippy::perf -W clippy::suspicious --deny=warnings
The contribution guide suggests running
clippy
. There are several pull requests that use thecargo clippy fix
. But at the same time, I did not find aci
job that would checkclippy
, which looks strange.The text was updated successfully, but these errors were encountered: