-
Notifications
You must be signed in to change notification settings - Fork 211
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
[Merged by Bors] - Remove in-code localDB migrations #5814
Conversation
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## develop #5814 +/- ##
=========================================
+ Coverage 80.2% 80.4% +0.1%
=========================================
Files 286 283 -3
Lines 29789 29319 -470
=========================================
- Hits 23910 23575 -335
+ Misses 4229 4150 -79
+ Partials 1650 1594 -56 ☔ View full report in Codecov by Sentry. |
node/node_version_check.go
Outdated
} | ||
|
||
// v1.5 requires going through v1.4 first as it removed in-code migrations. | ||
func verifyUpgradeToV15(cfg *config.Config) error { |
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 can probably be inlined, because if we require a higher version later we will just upgrade the version check on line 28 below instead of adding a new function.
bors merge |
## Motivation The code of migrations is not needed anymore, we will require to update to 1.4 before going to 1.5.
Pull request successfully merged into develop. Build succeeded: |
Motivation
The code of migrations is not needed anymore, we will require to update to 1.4 before going to 1.5.
Description
Removed in-code migrations for the local DB (migrating from files to DB and renaming the DB).
Test Plan
n/a
TODO