-
Notifications
You must be signed in to change notification settings - Fork 19
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
Feature/fork etrog #114
Feature/fork etrog #114
Conversation
Move isForced to batch level Update tests to fork id 6 fix header timestamp Implement forced batches logic Remove GER from batchData Set block gas limit
Store block info tree root at the end of tx processing
fix TX_GAS_LIMIT
add compute global index
…e-l2-block add flags to skipChangeL2Block & skipWriteBlockInfoRoot
Feature/ep l2txhash
PR review resolution
add tests different batches
update smt class with etrog storage changes
Update monorepo package
Update vm version
Quality Gate passedThe SonarCloud Quality Gate passed, but some issues were introduced. 26 New issues |
Increase block gas limit to 2**64
Feature/new l2hash
Quality Gate passedThe SonarCloud Quality Gate passed, but some issues were introduced. 26 New issues |
@@ -12,6 +12,7 @@ on: | |||
branches: | |||
- main | |||
- develop | |||
- feature/fork-etrog |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No need now
@@ -35,7 +37,7 @@ | |||
}, | |||
"homepage": "https://github.com/0xPolygonHermez/zkevm-commonjs#readme", | |||
"devDependencies": { | |||
"@0xpolygonhermez/zkevm-contracts": "github:0xPolygonHermez/zkevm-contracts#v3.0.0-rc.1-fork.6", | |||
"@0xpolygonhermez/zkevm-contracts": "github:0xPolygonHermez/zkevm-contracts#feature/l1-info-tree", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do we have tag?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
tests will fail since we need still to migrate to new bridge LxLy
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
💯
This PR does the following:
fork-etrog