Skip to content

Comments

Remove unreachable void pattern for ConnectionLimits#8871

Merged
mergify[bot] merged 1 commit intosigp:unstablefrom
jimmygchen:cleanup-unreachable-void-patterns
Feb 20, 2026
Merged

Remove unreachable void pattern for ConnectionLimits#8871
mergify[bot] merged 1 commit intosigp:unstablefrom
jimmygchen:cleanup-unreachable-void-patterns

Conversation

@jimmygchen
Copy link
Member

Description

Remove the #[allow(unreachable_patterns)] and unreachable BehaviourEvent::ConnectionLimits match arm. The MSRV is well above 1.82, so the compiler natively recognises that ConnectionLimits events are uninhabited (Infallible).

Closes #6356

@jimmygchen jimmygchen added ready-for-review The code is ready for review low-hanging-fruit Easy to resolve, get it before someone else does! labels Feb 20, 2026
@jimmygchen jimmygchen marked this pull request as ready for review February 20, 2026 03:54
@jimmygchen jimmygchen requested a review from jxs as a code owner February 20, 2026 03:54
Copy link
Member

@michaelsproul michaelsproul left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice 🔥

@michaelsproul michaelsproul added ready-for-merge This PR is ready to merge. and removed ready-for-review The code is ready for review labels Feb 20, 2026
@mergify mergify bot added the queued label Feb 20, 2026
@mergify
Copy link

mergify bot commented Feb 20, 2026

Merge Queue Status

Rule: default


This pull request spent 30 minutes 26 seconds in the queue, including 28 minutes 31 seconds running CI.

Required conditions to merge
  • check-success=local-testnet-success
  • check-success=test-suite-success

mergify bot added a commit that referenced this pull request Feb 20, 2026
@mergify mergify bot merged commit 8d4af65 into sigp:unstable Feb 20, 2026
36 checks passed
@mergify mergify bot removed the queued label Feb 20, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

low-hanging-fruit Easy to resolve, get it before someone else does! ready-for-merge This PR is ready to merge.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants