-
Notifications
You must be signed in to change notification settings - Fork 166
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
[Execution] state migration clean up #3192
[Execution] state migration clean up #3192
Conversation
storageUsedUpdateMigration.Migrate, | ||
mgr.PruneMigration, |
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.
We don't need these two if there is no other migration applied
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 no longer need the PruneMigration
If I remember correctly we always ran that.
Codecov Report
@@ Coverage Diff @@
## master #3192 +/- ##
==========================================
- Coverage 53.87% 51.74% -2.14%
==========================================
Files 743 615 -128
Lines 68520 58126 -10394
==========================================
- Hits 36917 30077 -6840
+ Misses 28557 25509 -3048
+ Partials 3046 2540 -506
Flags with carried forward coverage won't be shown. Click here to find out more.
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. |
storageUsedUpdateMigration.Migrate, | ||
mgr.PruneMigration, |
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 no longer need the PruneMigration
If I remember correctly we always ran that.
bors merge |
This PR remove the migration codes that has already been a applied on all networks.