Skip to content

feat: handle KnownChains::Gouda; pin signet-cold-sql via git-rev#149

Draft
rswanson wants to merge 6 commits into
mainfrom
feat/gouda-arm
Draft

feat: handle KnownChains::Gouda; pin signet-cold-sql via git-rev#149
rswanson wants to merge 6 commits into
mainfrom
feat/gouda-arm

Conversation

@rswanson
Copy link
Copy Markdown
Member

Summary

  • Adds KnownChains::Gouda => SlotCalculator::parmigiana_host() arm in src/utils/calc.rs (delegates to parmigiana host).
  • Pins signet-constants to signet-sdk gouda branch (rev 8a85a46).
  • Pins signet-cold-sql (plus transitive signet-cold, signet-storage-types) to storage gouda branch (rev 7841fab) to eliminate a diamond dep that surfaced downstream.

Test plan

  • cargo test --workspace --all-targets green
  • cargo fmt --check, cargo clippy -- -D warnings clean

Related branches

  • signet-sdk#236 (adds the Gouda variant being matched here)
  • storage feat/gouda-sdk-deps (provides the storage crates this branch pins)

🤖 Generated with Claude Code

rswanson and others added 6 commits May 17, 2026 19:23
Bump signet-constants to git-rev 8a85a4636bccf971226a9c80c311832ce33e998c
(v0.19.0, adds KnownChains::Gouda) and add the Gouda arm in the
From<KnownChains> for SlotCalculator impl, delegating to
parmigiana_host() since Gouda runs on the same host chain.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Replace crates.io signet-cold-sql v0.9.0 with a git-rev pin against
the signet-storage gouda branch (7841fabdd71dd5b58a7bd21ccd9b7d25d11ddc55)
to eliminate the diamond dep when consumed by node-components.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Update storage family pin from 7841fabd to 5ab8f75e to match the new
gouda SHA and resolve the diamond dependency with downstream consumers.
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
…6fa)

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
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.

1 participant