Skip to content

fix(rust): collapse match guard to fix clippy::collapsible_match CI failure#21

Merged
w-martin merged 4 commits into
mainfrom
worktree-agent-a8ec2ca0fb3645d90
May 26, 2026
Merged

fix(rust): collapse match guard to fix clippy::collapsible_match CI failure#21
w-martin merged 4 commits into
mainfrom
worktree-agent-a8ec2ca0fb3645d90

Conversation

@w-martin

Copy link
Copy Markdown
Owner

Fixes #20

The @stable Rust toolchain rolled to a Clippy version that enforces collapsible_match as a hard error. This is a zero-behaviour-change refactor: moves name.id.as_str() == \"concat\" from a nested if block into the match arm guard.

Unblocks all open PRs (#13#19) which currently fail CI at the Rust lint step.

Test plan

  • cargo clippy -- -D warnings passes in rust/
  • cargo test passes in rust/

🤖 Generated with Claude Code

w-martin and others added 3 commits May 24, 2026 12:33
Rustfmt at the current toolchain version collapses two-line if-let
conditions into a single line when they fit within the 100-char limit.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@codecov-commenter

Copy link
Copy Markdown

Welcome to Codecov 🎉

Once you merge this PR into your default branch, you're all set! Codecov will compare coverage reports and display results in all future pull requests.

ℹ️ You can also turn on project coverage checks and project coverage reporting on Pull Request comment

Thanks for integrating Codecov - We've got you covered ☂️

@w-martin w-martin merged commit 75a8c6b into main May 26, 2026
6 checks passed
@w-martin w-martin deleted the worktree-agent-a8ec2ca0fb3645d90 branch May 26, 2026 10:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

fix(rust): clippy::collapsible_match error in lib.rs blocks all CI

2 participants