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

feat(ethexe): add more accurate gas estimation for commitments #4484

Merged
merged 5 commits into from
Feb 4, 2025

Conversation

StackOverflowExcept1on
Copy link
Member

@StackOverflowExcept1on StackOverflowExcept1on commented Jan 30, 2025

Previously we could spend sequencer funds to send failed transaction. Now gas will be evaluated for everything except Gear.blockIsPredecessor().

Verified

This commit was signed with the committer’s verified signature.
tungbq Tung Leo
@StackOverflowExcept1on StackOverflowExcept1on added the A0-pleasereview PR is ready to be reviewed by the team label Jan 30, 2025

Verified

This commit was signed with the committer’s verified signature.
tungbq Tung Leo

Verified

This commit was signed with the committer’s verified signature.
tungbq Tung Leo

Verified

This commit was signed with the committer’s verified signature.
tungbq Tung Leo
@StackOverflowExcept1on StackOverflowExcept1on added E1-forcenatwin Force CI to run native Windows build in PR E3-forcerelease Force CI to run release build in PR labels Feb 4, 2025

Verified

This commit was signed with the committer’s verified signature.
tungbq Tung Leo
@StackOverflowExcept1on StackOverflowExcept1on merged commit 48e00ab into master Feb 4, 2025
14 checks passed
@StackOverflowExcept1on StackOverflowExcept1on deleted the av/gas-estimation branch February 4, 2025 13:42
breathx pushed a commit that referenced this pull request Feb 7, 2025

Verified

This commit was signed with the committer’s verified signature.
tungbq Tung Leo
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A0-pleasereview PR is ready to be reviewed by the team E1-forcenatwin Force CI to run native Windows build in PR E3-forcerelease Force CI to run release build in PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants