Skip to content

Cargo clippy#178

Merged
François Dubois (fdubois1) merged 2 commits intomasterfrom
cargo_clippy
Aug 7, 2021
Merged

Cargo clippy#178
François Dubois (fdubois1) merged 2 commits intomasterfrom
cargo_clippy

Conversation

@fdubois1
Copy link
Copy Markdown
Contributor

I ran cargo clippy and fixed warning

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Copy link
Copy Markdown
Member

@CBenoit Benoît Cortier (CBenoit) left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!
But something failed in CI

@awakecoding
Copy link
Copy Markdown
Contributor

Benoît Cortier (@CBenoit) inquiete toi pas pour le CI, on a eu pas mal de problemes cette semaine, je vais m'arranger avec 👍

@fdubois1 François Dubois (fdubois1) merged commit 846b874 into master Aug 7, 2021
@fdubois1 François Dubois (fdubois1) deleted the cargo_clippy branch August 7, 2021 20:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants