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.

Sign up for GitHub

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

Jump to bottom

Increase Holesky & Sepolia gaslimit by 66% to 60M #8263

Draft
wants to merge 4 commits into
base: master
Choose a base branch
from
Draft

Increase Holesky & Sepolia gaslimit by 66% to 60M #8263

wants to merge 4 commits into from

Conversation

benaadams
Copy link
Member

@benaadams benaadams commented Feb 24, 2025 •
edited
Loading

Changes

Types of changes

What types of changes does your code introduce?

  • Build-related changes

Testing

Requires testing

  • Yes

If yes, did you write tests?

  • Yes

MarekM25
MarekM25 reviewed Feb 24, 2025
View reviewed changes
Copy link
Contributor

@MarekM25 MarekM25 left a comment

Choose a reason for hiding this comment

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

What is the gas-limit without the issue with 10MB limits btw? I think it is above 60M but we have to calculate it

@kamilchodola
Copy link
Contributor

@benaadams add sepolia? :D Hopefully is not a bad omen.

@benaadams
Copy link
Member Author

@benaadams add sepolia? :D Hopefully is not a bad omen.

Added

@benaadams benaadams changed the title Increase Holesky gaslimit by 66% to 60M Increase Holesky & Sepolia gaslimit by 66% to 60M Feb 27, 2025
@LukaszRozmej
Copy link
Member

LukaszRozmej commented Feb 27, 2025 •
edited
Loading

IMO we shouldn't move it without more social consensus first. Which I expect we will get.

@benaadams
Copy link
Member Author

IMO we shouldn't move it without more social consensus first. Which I expect we will get.

Was agreed on ACDE 203

@benaadams
Copy link
Member Author

IMO we shouldn't move it without more social consensus first. Which I expect we will get.

Was agreed on ACDE 203

At this timestamp https://www.youtube.com/live/uh1hZCE4k0w?si=QyltoA2jf6FuYOyB&t=3344

@LukaszRozmej LukaszRozmej marked this pull request as draft February 27, 2025 12:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@MarekM25 MarekM25 MarekM25 left review comments

@ak88 ak88 ak88 approved these changes

@LukaszRozmej LukaszRozmej LukaszRozmej approved these changes

@smartprogrammer93 smartprogrammer93 smartprogrammer93 approved these changes

@kamilchodola kamilchodola kamilchodola approved these changes

@marcindsobczak marcindsobczak marcindsobczak approved these changes

Assignees
No one assigned
Labels
DO NOT MERGE performance is good
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
7 participants
@benaadams @kamilchodola @LukaszRozmej @ak88 @MarekM25 @smartprogrammer93 @marcindsobczak