-
Notifications
You must be signed in to change notification settings - Fork 410
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
Update to ibc-go v7 beta2 + protoVer=0.11.5 #1190
Conversation
Kudos, SonarCloud Quality Gate passed! |
Codecov Report
@@ Coverage Diff @@
## develop_sdk47 #1190 +/- ##
=================================================
- Coverage 59.30% 59.24% -0.06%
=================================================
Files 53 53
Lines 7237 7244 +7
=================================================
Hits 4292 4292
- Misses 2650 2657 +7
Partials 295 295
|
dude thank you! |
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.
Thanks a lot of the PR! 🥇
@@ -170,7 +170,7 @@ format: format-tools | |||
############################################################################### | |||
### Protobuf ### | |||
############################################################################### | |||
protoVer=0.11.2 | |||
protoVer=0.11.5 |
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.
👍
@@ -9,7 +9,7 @@ require ( | |||
github.com/cosmos/gogogateway v1.2.0 // indirect | |||
github.com/cosmos/gogoproto v1.4.3 | |||
github.com/cosmos/iavl v0.19.4 | |||
github.com/cosmos/ibc-go/v6 v6.0.0-20230124092939-cfcdec0b33a2 | |||
github.com/cosmos/ibc-go/v7 v7.0.0-beta2 |
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.
👍
@@ -423,3 +423,14 @@ func isSubset(super, sub []string) bool { | |||
} | |||
return matches == len(sub) | |||
} | |||
|
|||
// AllAuthorizedAddresses returns the list of authorized addresses. Can be empty. |
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 would be included in the #1189 backport. It is easier track changes separately for a clean history
Updated to use the newest ibc-go v7 beta tag and the latest version of the proto builder image.