Skip to content

Conversation

@kamuik16
Copy link
Contributor

Fixes #16847

Copy link
Collaborator

@mattsse mattsse left a comment

Choose a reason for hiding this comment

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

nice, smol nit

@github-project-automation github-project-automation bot moved this from Backlog to In Progress in Reth Tracker Jun 18, 2025
@mattsse
Copy link
Collaborator

mattsse commented Jun 18, 2025

can you also rebase main which should have a fix for the ci issues

@kamuik16
Copy link
Contributor Author

can you also rebase main which should have a fix for the ci issues

Done, and applied all the nits too.

@kamuik16 kamuik16 requested a review from mattsse June 18, 2025 12:56
Copy link
Collaborator

@mattsse mattsse left a comment

Choose a reason for hiding this comment

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

lgtm!

@mattsse mattsse enabled auto-merge June 18, 2025 13:07
@mattsse mattsse added this pull request to the merge queue Jun 18, 2025
Merged via the queue into paradigmxyz:main with commit 04f09f9 Jun 18, 2025
44 checks passed
@github-project-automation github-project-automation bot moved this from In Progress to Done in Reth Tracker Jun 18, 2025
@kamuik16 kamuik16 deleted the kamui/use-max-blobs-per-tx branch June 18, 2025 13:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

Use BlobParams.max_blobs_per_tx when validating 4844

2 participants