-
Notifications
You must be signed in to change notification settings - Fork 93
chore(deps): ibc-go to v10.1.1, wasmd 0.55.1 #272
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
chore(deps): ibc-go to v10.1.1, wasmd 0.55.1 #272
Conversation
|
closes #257 |
gjermundgaraba
left a comment
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.
Thank you! This mostly looks good, but needs a bit of cleanup to bring over the finish line
| // 2. 'auth' | ||
| // 3. 'ibc channel' | ||
| // 4. 'ibc port' | ||
| app.IBCKeeper = ibckeeper.NewKeeper( |
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.
The comment here was wrong, this is not the fee keeper
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.
weird, ibckeeper already existed:
https://github.com/cosmos/ibc-apps/pull/272/files?diff=unified#diff-c7aeadef5480ea593dd152867ab52c7f60e2df19c2ada3766951cd5f3d54bc1aR512
already exists before
Co-authored-by: Gjermund Garaba <[email protected]>
Co-authored-by: Gjermund Garaba <[email protected]>
No description provided.