Skip to content

Update and cleanup Electra preset #7303

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 1 commit into from
Apr 10, 2025
Merged

Update and cleanup Electra preset #7303

merged 1 commit into from
Apr 10, 2025

Conversation

@michaelsproul
Copy link
Member

@michaelsproul michaelsproul commented Apr 10, 2025

Proposed Changes

  • Update the bundled electra.yaml preset files for mainnet and minimal to match consensus-specs as of: ethereum/consensus-specs@bf09ede
  • Add the field max_pending_deposits_per_epoch to our runtime representation of ElectraPreset. This results in it appearing in /eth/v1/config/spec where it was previously absent.

Additional Info

I have not updated the Gnosis preset. That is happening as part of:

@michaelsproul michaelsproul added ready-for-review The code is ready for review electra Required for the Electra/Prague fork v7.0.0 New release c. Q1 2025 low-hanging-fruit Easy to resolve, get it before someone else does! labels Apr 10, 2025
macladson
macladson approved these changes Apr 10, 2025
View reviewed changes
Copy link
Member

@macladson macladson left a comment

Choose a reason for hiding this comment

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

LGTM!

@michaelsproul michaelsproul added ready-for-merge This PR is ready to merge. and removed ready-for-review The code is ready for review labels Apr 10, 2025
mergify bot added a commit that referenced this pull request Apr 10, 2025
@mergify
Merge of #7303
1902feb
@mergify mergify bot mentioned this pull request Apr 10, 2025
Closed
6 tasks
@mergify mergify bot merged commit bb5b00e into sigp:release-v7.0.0 Apr 10, 2025
31 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@macladson macladson macladson approved these changes

Assignees

No one assigned

Labels

electra Required for the Electra/Prague fork low-hanging-fruit Easy to resolve, get it before someone else does! ready-for-merge This PR is ready to merge. v7.0.0 New release c. Q1 2025

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@michaelsproul @macladson