-
-
Notifications
You must be signed in to change notification settings - Fork 289
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.
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
feat: move requests from execution payload to beacon block body #7094
Conversation
Performance Report✔️ no performance regression detected Full benchmark results
|
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## unstable #7094 +/- ##
============================================
- Coverage 50.89% 50.82% -0.07%
============================================
Files 595 597 +2
Lines 39652 39789 +137
Branches 2258 2055 -203
============================================
+ Hits 20182 20224 +42
- Misses 19470 19565 +95 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
* Move requests from payload to block body * Lint * Add execution requests to engine api * Remove engine_getPayloadBodies*V2 * Update spec test version * Lint * Fix unit test and polish * Remove todo
Changes
Containers
ExecutionRequests
to encapsulate the three requestsExecutionRequests
to beacon block bodyAPIs
engine_newPayloadV4
andengine_getPayloadV4
to include execution requestsengine_getPayloadBodiesByRangeV2
andengine_getPayloadBodiesByHashV2
Light Client
Spec
Waiting for
next spec test releaseand ethereum/execution-apis#587 to be finalizedThis PR does not contain update to the builder flow. Will update it in the next PR.