-
Notifications
You must be signed in to change notification settings - Fork 3.6k
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
build(deps): bump to cosmossdk.io/api v0.7.3 #19376
Conversation
I guess we need to fix this first:
|
EDIT: because we want this to be fixed sooner than later (in v0.50.4), and cosmos/cosmos-proto#131 takes longer than expected. I'll make a cosmossdk.io/api v0.7.3 cherry-picking this change, so we don't need to bump api to latest. |
ba4eb9b
to
936c43e
Compare
go.mod
Outdated
@@ -3,7 +3,7 @@ go 1.21 | |||
module github.com/cosmos/cosmos-sdk | |||
|
|||
require ( | |||
cosmossdk.io/api v0.7.2 | |||
cosmossdk.io/api v0.7.3-0.20240216161322-c0ecaf7cf2f1 |
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.
Description
Closes: #19374
Attempt to use the latest api from main in release/v0.50.x to fix the issue where api v0.7.2 didn't contain the released changes from gov.
We should still verift how autocli reacts given that we will be adding unreleased things into api (normally this #18324 should nicely hide those commands, but let's see).
Once I confirm everything works in CI and locally, we can tag api v0.7.3 and bump it here.
Author Checklist
All items are required. Please add a note to the item if the item is not applicable and
please add links to any relevant follow up issues.
I have...
!
in the type prefix if API or client breaking changeCHANGELOG.md
Reviewers Checklist
All items are required. Please add a note if the item is not applicable and please add
your handle next to the items reviewed if you only reviewed selected items.
I have...