-
Notifications
You must be signed in to change notification settings - Fork 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
cleanup: delete l2geth-exporter #5932
Conversation
Removes the `l2geth-exporter` package from the monorepo. This exposed metrics for the legacy system and is no longer necessary.
|
✅ Deploy Preview for opstack-docs canceled.
|
Codecov Report
Additional details and impacted files@@ Coverage Diff @@
## develop #5932 +/- ##
===========================================
+ Coverage 43.03% 43.05% +0.01%
===========================================
Files 477 477
Lines 30723 30723
Branches 877 877
===========================================
+ Hits 13222 13227 +5
Misses 16472 16472
+ Partials 1029 1024 -5
Flags with carried forward coverage won't be shown. Click here to find out more. |
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.
Looks good, but I think @protolambda wanted to copy the entire monorepo to a new repo to isolate + freeze pre-bedrock code. That should probably be done before this is merged.
Update: this will be part of https://linear.app/optimism/issue/CLI-4053/monorepo-post-migration-cleanup |
Description
Removes the
l2geth-exporter
package from the monorepo. This exposed metrics for the legacy system and is no longer necessary.