Skip to content

fix: nightly clippy fixes#1085

Merged
zerosnacks merged 3 commits intomainfrom
zerosnacks/fix-unused-likely-unlikely-feature
Mar 16, 2026
Merged

fix: nightly clippy fixes#1085
zerosnacks merged 3 commits intomainfrom
zerosnacks/fix-unused-likely-unlikely-feature

Conversation

@decofe
Copy link
Contributor

@decofe decofe commented Mar 16, 2026

Summary

Fix two nightly clippy errors that break CI.

Changes

  • primitives: remove unused likely_unlikely feature gate — the intrinsics are now available under core_intrinsics (already declared)
  • dyn-abi: collapse if into outer match arm in eip712 parser (collapsible_match lint)

Co-Authored-By: zerosnacks 95942363+zerosnacks@users.noreply.github.com

Prompted by: zerosnacks

The likely_unlikely feature was stabilized under core_intrinsics,
which is already declared. The separate feature gate is no longer
needed and causes a clippy error on newer nightly.

Co-Authored-By: zerosnacks <95942363+zerosnacks@users.noreply.github.com>
decofe and others added 2 commits March 16, 2026 07:53
Co-Authored-By: zerosnacks <95942363+zerosnacks@users.noreply.github.com>
Co-Authored-By: zerosnacks <95942363+zerosnacks@users.noreply.github.com>
@decofe decofe changed the title fix(primitives): remove unused likely_unlikely feature gate fix: nightly clippy fixes Mar 16, 2026
@zerosnacks zerosnacks marked this pull request as ready for review March 16, 2026 08:15
@zerosnacks zerosnacks self-assigned this Mar 16, 2026
@github-project-automation github-project-automation bot moved this to Reviewed in Alloy Mar 16, 2026
@zerosnacks zerosnacks merged commit 5f451a5 into main Mar 16, 2026
31 checks passed
@zerosnacks zerosnacks deleted the zerosnacks/fix-unused-likely-unlikely-feature branch March 16, 2026 11:11
@github-project-automation github-project-automation bot moved this from Reviewed to Done in Alloy Mar 16, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

3 participants