Skip to content

Add randomized execution requests in random_block_electra#4032

Merged
jtraglia merged 7 commits into
ethereum:devfrom
KatyaRyazantseva:random-block-electra
Dec 4, 2024
Merged

Add randomized execution requests in random_block_electra#4032
jtraglia merged 7 commits into
ethereum:devfrom
KatyaRyazantseva:random-block-electra

Conversation

@KatyaRyazantseva

Copy link
Copy Markdown
Contributor

This PR adds randomized BeaconBlock.body.execution_requests in random_block_electra.

@jtraglia jtraglia left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Looks good! A little bug & some nits. I would probably delete some of the whitespace but that's just my personal style.

Comment thread tests/core/pyspec/eth2spec/test/helpers/multi_operations.py Outdated
Comment thread tests/core/pyspec/eth2spec/test/helpers/multi_operations.py Outdated
Comment thread tests/core/pyspec/eth2spec/test/helpers/multi_operations.py
Comment thread tests/core/pyspec/eth2spec/test/helpers/multi_operations.py
KatyaRyazantseva and others added 2 commits December 3, 2024 16:36
Co-authored-by: Justin Traglia <95511699+jtraglia@users.noreply.github.com>
@KatyaRyazantseva KatyaRyazantseva marked this pull request as ready for review December 4, 2024 03:13

@jtraglia jtraglia left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM. Thank you!

@jtraglia jtraglia merged commit 72061bb into ethereum:dev Dec 4, 2024
etan-status added a commit to etan-status/consensus-specs that referenced this pull request Jan 6, 2025
New tests were added in ethereum#4032 with incorrect EL block hash, fix these.
etan-status added a commit to etan-status/consensus-specs that referenced this pull request Jan 6, 2025
New tests were added in ethereum#4032 with incorrect EL block hash, fix these.
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.

2 participants