Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Fix lints for Rust 1.84 #638

Merged
merged 6 commits into from
Jan 9, 2025
Merged

Conversation

smalis-msft
Copy link
Contributor

The biggest two here are changing map_or(false, to is_some_and or is_ok_and, and the changing of rules around how refs in patterns work (which is technically a future-compat lint, but its easy enough to take now), but there's also some elided lifetimes and other general cleanups.

@smalis-msft smalis-msft requested review from a team as code owners January 9, 2025 18:06
@smalis-msft smalis-msft enabled auto-merge (squash) January 9, 2025 18:08
@smalis-msft smalis-msft merged commit 8d9d904 into microsoft:main Jan 9, 2025
25 checks passed
@smalis-msft smalis-msft deleted the rust-184-lints branch January 9, 2025 20:41
benhillis pushed a commit to benhillis/openvmm that referenced this pull request Jan 15, 2025
The biggest two here are changing `map_or(false,` to `is_some_and` or
`is_ok_and`, and the changing of rules around how refs in patterns work
(which is technically a future-compat lint, but its easy enough to take
now), but there's also some elided lifetimes and other general cleanups.
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants