Skip to content
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 #178

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Fix clippy #178

wants to merge 1 commit into from

Conversation

uggla
Copy link
Collaborator

@uggla uggla commented Jun 9, 2022

Hi @bpetit,

A small MR to:

  • Fix all clippy warnings from Rust 1.61.

Hoping you will have time to review this.

@uggla uggla force-pushed the edition_2021_+_clippy branch from 95b7489 to 2daaa30 Compare June 25, 2022 14:16
@uggla uggla changed the title Edition 2021 + clippy Fix clippy Jun 25, 2022
- Remove all clippy warnings using Rust release 1.61.
@uggla uggla force-pushed the edition_2021_+_clippy branch from 2daaa30 to 2df4dcf Compare June 25, 2022 14:23
@bpetit
Copy link
Contributor

bpetit commented Jul 13, 2022

Hi @uggla ! Thanks a lot for the PR.

I think we already fixed this in the ongoing PRs on dev, so it should be fixed with the release 0.5

If not, I'll merge your PR to fix the remaining errors.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: To do
Development

Successfully merging this pull request may close these issues.

2 participants