-
Notifications
You must be signed in to change notification settings - Fork 983
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
migration fixes #4135
base: main
Are you sure you want to change the base?
migration fixes #4135
Conversation
ah, an integration test is failing |
0c27441
to
4c20ff8
Compare
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #4135 +/- ##
==========================================
- Coverage 74.56% 74.51% -0.05%
==========================================
Files 341 341
Lines 107495 107638 +143
==========================================
+ Hits 80152 80206 +54
- Misses 27343 27432 +89 ☔ View full report in Codecov by Sentry. |
4c20ff8
to
c1f2b14
Compare
@tzemanovic what's the latest with this branch? |
it's rdy to merge whenever we want to include it |
Describe your changes
skips writing diffs for migration to close Invalid MerkleTreeStoresRead after restarting a 0.45 to 0.46 upgraded validator #4112 (extended e2e migration test to cover this)Checklist before merging
breaking::
labelsnamada-docs
reponamada-indexer
ornamada-masp-indexer
, a corresponding PR is opened in that repo