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

[Jovian] Add Jovian activation #13722

Draft
wants to merge 1 commit into
base: develop
Choose a base branch
from
Draft

[Jovian] Add Jovian activation #13722

wants to merge 1 commit into from

Conversation

mdehoog
Copy link
Contributor

@mdehoog mdehoog commented Jan 13, 2025

Description

Adds the required code for activating the Jovian hard fork. Doing this in a separate PR allows Jovian implementors to build on top of it.

Built on top of #13549.
Prerequisite for #13539.
Requires ethereum-optimism/superchain-registry#827, ethereum-optimism/op-geth#463.

Tests

Added tests in relevant spots for hard fork activation.

Copy link

codecov bot commented Jan 13, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 78.65%. Comparing base (2fb8c66) to head (b401a66).

Additional details and impacted files
@@             Coverage Diff              @@
##           develop   #13722       +/-   ##
============================================
+ Coverage    47.25%   78.65%   +31.39%     
============================================
  Files          958      166      -792     
  Lines        80075     9410    -70665     
  Branches       773      773               
============================================
- Hits         37836     7401    -30435     
+ Misses       39335     1752    -37583     
+ Partials      2904      257     -2647     
Flag Coverage Δ
cannon-go-tests-32 62.26% <ø> (-1.98%) ⬇️
cannon-go-tests-64 57.35% <ø> (-1.62%) ⬇️
contracts-bedrock-tests 90.57% <ø> (-0.20%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

see 806 files with indirect coverage changes

@mdehoog mdehoog changed the title [Jovian] Add Jovian activation code [Jovian] Add Jovian activation Jan 13, 2025
replace github.com/ethereum/go-ethereum => github.com/ethereum-optimism/op-geth v1.101411.5-rc.1
replace github.com/ethereum/go-ethereum => github.com/mdehoog/op-geth v0.0.0-20250113013512-5d39fcd5b165

replace github.com/ethereum-optimism/superchain-registry/superchain => github.com/mdehoog/superchain-registry/superchain v0.0.0-20250112212108-52eef42cce64
Copy link
Contributor Author

Choose a reason for hiding this comment

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

@@ -253,7 +253,9 @@ require (
rsc.io/tmplfunc v0.0.3 // indirect
)

replace github.com/ethereum/go-ethereum => github.com/ethereum-optimism/op-geth v1.101411.5-rc.1
replace github.com/ethereum/go-ethereum => github.com/mdehoog/op-geth v0.0.0-20250113013512-5d39fcd5b165
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant