-
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
x:ibc: remove alias.go usage #6429
Conversation
Codecov Report
@@ Coverage Diff @@
## master #6429 +/- ##
=======================================
Coverage 55.72% 55.72%
=======================================
Files 465 465
Lines 27782 27782
=======================================
Hits 15481 15481
Misses 11197 11197
Partials 1104 1104 |
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.
nice work, no need to keep the number in the package import name for IBC submodules
simapp/app.go
Outdated
@@ -30,6 +30,8 @@ import ( | |||
transfer "github.com/cosmos/cosmos-sdk/x/ibc-transfer" | |||
ibcclient "github.com/cosmos/cosmos-sdk/x/ibc/02-client" | |||
port "github.com/cosmos/cosmos-sdk/x/ibc/05-port" | |||
ibc24host "github.com/cosmos/cosmos-sdk/x/ibc/24-host" |
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.
ibc24host
can just be host
unless it conflicts with something (which I don't think it should) then it would be ibchost
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.
LGTM
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.
LGTM
@Mergifyio refresh |
Command |
* x:ibc: remove alias.go usage * Replace ibc24host with ibchost
Description
This PR closes a sub task in #6311
Before we can merge this PR, please make sure that all the following items have been
checked off. If any of the checklist items are not applicable, please leave them but
write a little note why.
docs/
) or specification (x/<module>/spec/
)godoc
comments.Unreleased
section inCHANGELOG.md
Files changed
in the Github PR explorer