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.

Sign up for GitHub

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

Jump to bottom

make cargo clippy pass. #592

Merged

jacksonrnewhouse merged 1 commit into master from clippy_compliance

Apr 22, 2024

Contributor

jacksonrnewhouse commented Apr 22, 2024 •

edited

Loading

cargo clippy now passes. I started with cargo clippy --fix and then did manual fixes for the rest. The main things I fixed manually:

Switch single matches to if let blocks.
Change &Vec<> function args to &[].
Define some complex types.
Add allows where it made sense.


          make cargo clippy pass.

0c4cb5a

jacksonrnewhouse requested a review from mwylde

April 22, 2024 18:56

mwylde approved these changes

View reviewed changes

jacksonrnewhouse merged commit 3cab907 into master

6 checks passed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet