Skip to content

Fix many issues reported by Clippy #211

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom
Closed
wants to merge 1 commit into from
Closed

Fix many issues reported by Clippy #211

wants to merge 1 commit into from

Conversation

andrewliebenow
Copy link
Contributor

No description provided.

@andrewliebenow andrewliebenow mentioned this pull request Sep 16, 2024
Closed
@andrewliebenow
Copy link
Contributor Author

Note that anyone using an editor set up to run cargo clippy instead of cargo check will still get many errors/warnings in their editor (although many fewer than before this PR), because this PR does not include the changes to the Cargo.toml files that were in #207

@jgarzik jgarzik mentioned this pull request Sep 16, 2024
Merged
@jgarzik
Copy link
Contributor

jgarzik commented Sep 16, 2024

Cherry-picked most of these into #213 for merging.

@andrewliebenow andrewliebenow mentioned this pull request Oct 12, 2024
Merged
# for free to join this conversation on GitHub. Already have an account? # to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@andrewliebenow @jgarzik