-
Notifications
You must be signed in to change notification settings - Fork 3.3k
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
add new UpdateType & fields to L1 system config required for the Ecotone upgrade #8723
add new UpdateType & fields to L1 system config required for the Ecotone upgrade #8723
Conversation
Codecov ReportAttention:
Additional details and impacted files@@ Coverage Diff @@
## develop #8723 +/- ##
===========================================
+ Coverage 34.90% 34.94% +0.04%
===========================================
Files 165 165
Lines 7083 7089 +6
Branches 1193 1193
===========================================
+ Hits 2472 2477 +5
- Misses 4459 4460 +1
Partials 152 152
Flags with carried forward coverage won't be shown. Click here to find out more.
|
We are going to need unit tests:
For 2 + 3, they could be fuzz tests |
ba005f1
to
cf07d83
Compare
WalkthroughWalkthroughThe Changes
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media? TipsChatThere are 3 ways to chat with CodeRabbit's AI:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (invoked as PR comments)
Additionally, you can add CodeRabbit Configration File (
|
7333d86
to
20310db
Compare
Can you also add tests for the initialize case? It is very important that the blob scalar is 0 and the basefee scalar is the scalar. For both event emission and initial values |
2e7bbfa
to
2cef5d3
Compare
237d0a3
to
1b23164
Compare
1b23164
to
20310db
Compare
20310db
to
89f3e9c
Compare
a38a8bf
to
da63bf0
Compare
b37c865
Description
Ecotone upgrade requires new scalar fields to be added and support to update them.
Corresponding spec update: #8708