Skip to content

feat(github): Add zkevm 60M, 90M, 120M #1655

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
Merged
merged 4 commits into from
May 26, 2025
Merged

feat(github): Add zkevm 60M, 90M, 120M #1655

merged 4 commits into from
May 26, 2025

Conversation

marioevz
Copy link
Member

🗒️ Description

Add features:

  • zkEVM with 60 million block gas limit
  • zkEVM with 90 million block gas limit
  • zkEVM with 120 million block gas limit

🔗 Related Issues

None

✅ Checklist

  • All: Set appropriate labels for the changes.
  • All: Considered squashing commits to improve commit history.
  • All: Added an entry to CHANGELOG.md.
  • All: Considered updating the online docs in the ./docs/ directory.

@marioevz marioevz added the scope:ci Scope: Continuous Integration label May 26, 2025
jsign
jsign approved these changes May 26, 2025
View reviewed changes
kevaundray
kevaundray approved these changes May 26, 2025
View reviewed changes
Copy link
Contributor

@kevaundray kevaundray left a comment

Choose a reason for hiding this comment

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

LGTM.

Maybe we should be explicit on all of them, so zkevm -> zkevm_36M -- It would then be upto the caller to know that the current gas limit is 36M

@marioevz
Copy link
Member Author

LGTM.

Maybe we should be explicit on all of them, so zkevm -> zkevm_36M -- It would then be upto the caller to know that the current gas limit is 36M

Thanks for the suggestion, updated.

@marioevz marioevz merged commit 3810d22 into main May 26, 2025
25 checks passed
@marioevz marioevz deleted the add-zvevm-60m-90m-120m-features branch May 26, 2025 17:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jsign jsign jsign approved these changes

@kevaundray kevaundray kevaundray approved these changes

Assignees
No one assigned
Labels
scope:ci Scope: Continuous Integration
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@marioevz @jsign @kevaundray