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

[Merged by Bors] - New rust lints for rustc 1.64.0 #3602

Closed

Conversation

divagant-martian
Copy link
Collaborator

@divagant-martian divagant-martian commented Sep 22, 2022

Issue Addressed

fixes lints from the last rust release

Proposed Changes

Fix the lints, most of the lints by clippy::question-mark are false positives in the form of rust-lang/rust-clippy#9518 so it's allowed for now

Additional Info

@divagant-martian divagant-martian changed the title New rust lints New rust lints for rustc 1.64.0 Sep 22, 2022
@michaelsproul michaelsproul added ready-for-merge This PR is ready to merge. v3.1.2 Release after v3.1.0 (formerly v3.1.1) labels Sep 23, 2022
@paulhauner
Copy link
Member

bors r+

bors bot pushed a commit that referenced this pull request Sep 23, 2022
## Issue Addressed
fixes lints from the last rust release

## Proposed Changes
Fix the lints, most of the lints by `clippy::question-mark` are false positives in the form of rust-lang/rust-clippy#9518 so it's allowed for now

## Additional Info
@bors bors bot changed the title New rust lints for rustc 1.64.0 [Merged by Bors] - New rust lints for rustc 1.64.0 Sep 23, 2022
@bors bors bot closed this Sep 23, 2022
Woodpile37 pushed a commit to Woodpile37/lighthouse that referenced this pull request Jan 6, 2024
## Issue Addressed
fixes lints from the last rust release

## Proposed Changes
Fix the lints, most of the lints by `clippy::question-mark` are false positives in the form of rust-lang/rust-clippy#9518 so it's allowed for now

## Additional Info
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ready-for-merge This PR is ready to merge. v3.1.2 Release after v3.1.0 (formerly v3.1.1)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants