Skip to content

Fulu support#131

Merged
olegshmuelov merged 2 commits into
mainfrom
fulu-support
Sep 22, 2025
Merged

Fulu support#131
olegshmuelov merged 2 commits into
mainfrom
fulu-support

Conversation

@olegshmuelov
Copy link
Copy Markdown
Contributor

No description provided.

Copilot AI review requested due to automatic review settings September 17, 2025 13:40
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR adds support for the "Fulu" Ethereum upgrade by integrating the new fork into the beacon block signing functionality and adding corresponding test data for various block types.

  • Added case handling for spec.DataVersionFulu in both blinded and regular beacon block signing functions
  • Updated the go-eth2-client dependency to a pre-release version that includes Fulu support
  • Added comprehensive test data files for all supported block types in the Fulu format

Reviewed Changes

Copilot reviewed 12 out of 16 changed files in this pull request and generated no comments.

File Description
signer/sign_blinded_beacon_block.go Added Fulu case to handle new fork version in blinded beacon block signing
signer/sign_beacon_block.go Added Fulu case to handle new fork version in regular beacon block signing
go.mod Updated dependencies to support Fulu upgrade with pre-release go-eth2-client version
signer/testdata/blocks/*.json Added test data files for various block types (phase0, altair, bellatrix, capella, deneb, electra) to support testing

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@olegshmuelov olegshmuelov merged commit 4f73647 into main Sep 22, 2025
4 checks passed
@olegshmuelov olegshmuelov deleted the fulu-support branch September 22, 2025 11:31
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.

5 participants