Skip to content

Clippy fixes #3865

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
Aug 24, 2021
Merged

Clippy fixes #3865

merged 1 commit into from
Aug 24, 2021

Conversation

nipunn1313
Copy link
Contributor

This fix came up when running clippy with nightly. (I was using default nightly toolchain on my machine).

I noticed CI runs with stable clippy - and I added a commit to switch back to continually using stable clippy, as the comment seemed to imply that's what you wanted. I'd understand if you'd want to keep it pinned - let me know and I'll revert the change to ci/

@rust-highfive
Copy link

Thanks for the pull request, and welcome! The Rust team is excited to review your changes, and you should hear from @carols10cents (or someone else) soon.

Please see the contribution instructions for more information.

@nipunn1313
Copy link
Contributor Author

Spotted #3849 - so I'll remove that from this diff to keep things separate.

@Turbo87 Turbo87 added A-backend ⚙️ C-internal 🔧 Category: Nonessential work that would make the codebase more consistent or clear labels Aug 24, 2021
@Turbo87
Copy link
Member

Turbo87 commented Aug 24, 2021

thanks! :)

@bors r+

@bors
Copy link
Contributor

bors commented Aug 24, 2021

📌 Commit c170c24 has been approved by Turbo87

@bors
Copy link
Contributor

bors commented Aug 24, 2021

⌛ Testing commit c170c24 with merge f2f15a2...

@bors
Copy link
Contributor

bors commented Aug 24, 2021

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

@bors bors merged commit f2f15a2 into rust-lang:master Aug 24, 2021
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.

5 participants