Skip to content

Fix clippy warnings #4551

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

Merged
merged 1 commit into from
Feb 11, 2022
Merged

Fix clippy warnings #4551

merged 1 commit into from
Feb 11, 2022

Conversation

nightkr
Copy link
Contributor

@nightkr nightkr commented Feb 11, 2022

Split out from #4548

@Turbo87
Copy link
Member

Turbo87 commented Feb 11, 2022

I'm a little confused about this, because we're running clippy in CI (see https://github.com/rust-lang/crates.io/blob/master/.github/workflows/ci.yml#L66) and it does not appear to error on these issues. any clue why they show up for you but not in CI?

@Turbo87 Turbo87 added A-backend ⚙️ C-internal 🔧 Category: Nonessential work that would make the codebase more consistent or clear labels Feb 11, 2022
@nightkr
Copy link
Contributor Author

nightkr commented Feb 11, 2022

It might be because I'm running rustc 1.59.0-beta.6 (due to rust-lang/rust#91598).

@Turbo87
Copy link
Member

Turbo87 commented Feb 11, 2022

I see, thanks

@bors r+

@bors
Copy link
Contributor

bors commented Feb 11, 2022

📌 Commit d539ba9 has been approved by Turbo87

@bors
Copy link
Contributor

bors commented Feb 11, 2022

⌛ Testing commit d539ba9 with merge fb26ee6...

@bors
Copy link
Contributor

bors commented Feb 11, 2022

☀️ Test successful - checks-actions
Approved by: Turbo87
Pushing fb26ee6 to master...

@bors bors merged commit fb26ee6 into rust-lang:master Feb 11, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-backend ⚙️ C-internal 🔧 Category: Nonessential work that would make the codebase more consistent or clear
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants