Skip to content
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

Capture complete EIP-4844 blob tx with side car blobs on EL #188

Closed
wants to merge 8 commits into from

Conversation

noslav
Copy link
Member

@noslav noslav commented Mar 13, 2024

No description provided.

@noslav noslav changed the title Support BSP export for EIP-4844 blob tx Capture complete EIP-4844 blob tx with side car blobs on EL Mar 13, 2024
Copy link

codecov bot commented Mar 26, 2024

Codecov Report

Attention: Patch coverage is 0% with 42 lines in your changes are missing coverage. Please review.

Project coverage is 57.30%. Comparing base (5b4667c) to head (102abbd).

❗ Current head 102abbd differs from pull request most recent head 26e28f5. Consider uploading reports for the commit 26e28f5 to get more accurate results

Additional details and impacted files

Impacted file tree graph

@@             Coverage Diff             @@
##           develop     #188      +/-   ##
===========================================
- Coverage    57.35%   57.30%   -0.05%     
===========================================
  Files          600      600              
  Lines        80100    80126      +26     
===========================================
- Hits         45939    45920      -19     
- Misses       30275    30331      +56     
+ Partials      3886     3875      -11     
Files Coverage Δ
core/types/block_export.go 0.00% <0.00%> (ø)

... and 14 files with indirect coverage changes


Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 5b4667c...26e28f5. Read the comment docs.

Signed-off-by: Pranay Valson <[email protected]>
@noslav
Copy link
Member Author

noslav commented Jun 11, 2024

Closing in favor already merged PR #198

@noslav noslav closed this Jun 11, 2024
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