Skip to content

Bump Rust MSRV to 1.88, disabling clippy::collapsible_if - #2253

Open
ids1024 wants to merge 2 commits into
Smithay:masterfrom
ids1024:rust-1.88-min
Open

ids1024 wants to merge 2 commits into
Smithay:masterfrom
ids1024:rust-1.88-min

Conversation

@ids1024

@ids1024 ids1024 commented Oct 30, 2026

Copy link
Copy Markdown
Member

Description

Bumping from 1.87 to 1.88 provides if let chains. If there isn't anything specific holding back an MSRV bump, we might as well make use of that.

Clippy produces a lot of lints with this. It can fix them automatically, but produces a large diff. For now, this disables the clippy::collapsible_if lint. We can potentially enable that on a per-module basis so this later to not cause quite so many conflicts with other PRs.

Lints applied with cargo +nightly clippy --fix --all and formatted with cargo +nightly fmt.

Checklist

Bumping from 1.87 to 1.88 provides `if let` chains. If there isn't
anything specific holding back an MSRV bump, we might as well make use
of that.

Clippy produces a lot of lints with this. It can fix them automatically,
but produces a large diff. For now, this disables the
`clippy::collapsible_if` lint. We can potentially enable that on a
per-module basis so this later to not cause quite so many conflicts with
other PRs.

Lints applied with `cargo +nightly clippy --fix --all` and formatted
with `cargo +nightly fmt`.

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

1 participant