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

Update EIP-4844: Update blob base fee to base fee per blob gas #8316

Merged
merged 2 commits into from
Mar 29, 2024

Conversation

g11tech
Copy link
Contributor

@g11tech g11tech commented Mar 14, 2024

  • The blob base fee refereed in the EIP is actually base fee per blob gas, and seems to have confused quite a number of readers.
  • changes data fee references to blob fee (calc_data_fee => calc_bob_fee) going with the previous reasoning of renaming data => bob to not conflict with any future other "data" objects that can be added

This EIP updates the same since it doesn't has any material change since no header fields are affected (and hence doesn't affect consensus), just internal helper functions and constants which clients can update at their own pace

@github-actions github-actions bot added c-update Modifies an existing proposal t-core labels Mar 14, 2024
@eth-bot
Copy link
Collaborator

eth-bot commented Mar 14, 2024

✅ All reviewers have approved.

@eth-bot eth-bot added the a-review Waiting on author to review label Mar 14, 2024
@g11tech g11tech marked this pull request as ready for review March 21, 2024 14:12
@g11tech g11tech requested a review from eth-bot as a code owner March 21, 2024 14:12
@g11tech g11tech changed the title Update EIP-4844: Update blob base fee to blob base fee per gas Update EIP-4844: Update blob base fee to base fee per blob gas Mar 21, 2024
@eth-bot eth-bot enabled auto-merge (squash) March 29, 2024 19:57
Copy link
Collaborator

@eth-bot eth-bot left a comment

Choose a reason for hiding this comment

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

All Reviewers Have Approved; Performing Automatic Merge...

@eth-bot eth-bot merged commit 65f6bd6 into ethereum:master Mar 29, 2024
10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
a-review Waiting on author to review c-update Modifies an existing proposal t-core
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants