-
Notifications
You must be signed in to change notification settings - Fork 556
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
SDK v50 Upgrade - Cherry-Picks to Fork #8240
Comments
6 tasks
This was referenced May 30, 2024
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
From pre v0.47:
Features:
From post v0.47:
More CI
More CI
Consider adding test backinto integration
For the IAVL wrapper (I dont think this one is needed)
No longer needed now that Balances is used
Make sure slashing migration changes don't need to be backported (they do, this is where we change key)
Its unclear to me if these wrapper changes are needed, we don't use cometbft-db anymore and use cosmos-db
Questionable PRs After First Review
Skipping this one because denomOwners looks to be tracked in its own way now
Skipping this one because we don't have the option to get a param as a byte slice anymore
Might need this one when I run e2e in CI
Might need this one when I run e2e in CI
Its unclear to me that this speedup was already implemented in some other way, need Dev to look
Double check that no in-place-testnet changes were made that weren't upstreamed
For the IAVL wrapper (I dont think this one is needed)
No longer needed now that Balances is used
Its unclear to me if these wrapper changes are needed, we don't use cometbft-db anymore and use cosmos-db
Consider adding test backinto integration
The text was updated successfully, but these errors were encountered: