Conversation
|
Correcting my own PR description before anyone spends review time on it. I claimed this lint would turn CI red once runners picked up a newer stable. That's backwards. Your runners are already on rustc 1.98.1 and clippy there does not flag these blocks — #591 carries the unmodified So there is no CI breakage here, and the only thing this PR offers is the collapse itself. I've rewritten the description to say that. Happy to close it if it isn't worth the churn. |
Signed-off-by: Menglin Li <limenglin5911@gmail.com>
cadba42 to
5ddc284
Compare
|
Closing this — as noted above, it isn't a CI fix: clippy on the toolchain your CI runs (1.98.1) doesn't flag these blocks, and the collapse on its own isn't worth the review time. Leaving it here in case the lint comes back on a future stable. Sorry for the noise. |
Corrected after checking CI — the original description of this PR had the motivation backwards. See the note below.
clippy::collapsible_else_iffires on two blocks incrates/utopia-store/src/temporal.rsunder clippy 1.93.0:It does not fire on the toolchain CI actually runs (rustc 1.98.1, 2026-09-01) —
#591, which carries the unmodified file, passescargo clippy --workspace --all-targets -- -D warningson your runners. So this is not a CI fix, and nothing ondevis red because of it.What's left is a small readability change that some clippy versions ask for: an
elsewhose whole body is oneifreads as anelse if.-18/+22, behaviour unchanged.Entirely reasonable to close this as cosmetic churn — no argument from me if you'd rather not take it.
Checked locally (macOS, rustc 1.93.0):
cargo fmt --all --checkcargo clippy --workspace --all-targets -- -D warnings— cleancargo test -p utopia-storewithUTOPIA_DATABASE_URLset — 178 passed, 0 failed