Skip to content

ci: fix Foundry nightly lint failures#129

Merged
grandizzy merged 1 commit into
masterfrom
centaur/fix-foundry-nightly-lints-1783950578
Jul 13, 2026
Merged

ci: fix Foundry nightly lint failures#129
grandizzy merged 1 commit into
masterfrom
centaur/fix-foundry-nightly-lints-1783950578

Conversation

@decofe

@decofe decofe commented Jul 13, 2026

Copy link
Copy Markdown
Member

Fixes CI against the latest Foundry nightly after foundry-rs/foundry#15707:

  • uses bytes.concat for framed WebAuthn/RLP concatenation, avoiding conservative abi.encodePacked collision diagnostics on nightly without introducing nightly-only suppressions that stable Forge rejects
  • validates block.chainid fits uint64 before the intentional EIP-1559/EIP-7702 narrowing casts

Validated with stable Forge and latest nightly:

  • forge fmt --check
  • forge build --skip test --deny warnings
  • forge test

Prompted by: @grandizzy

@mablr mablr left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

LGTM

@grandizzy
grandizzy enabled auto-merge (squash) July 13, 2026 13:52
@grandizzy
grandizzy merged commit 6c27c5c into master Jul 13, 2026
22 checks passed
@grandizzy
grandizzy deleted the centaur/fix-foundry-nightly-lints-1783950578 branch July 13, 2026 13:53
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.

3 participants