-
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
deps: fix iavl revision #6492
deps: fix iavl revision #6492
Conversation
@@ -37,7 +37,7 @@ require ( | |||
github.com/tendermint/btcd v0.1.1 | |||
github.com/tendermint/crypto v0.0.0-20191022145703-50d29ede1e15 | |||
github.com/tendermint/go-amino v0.15.1 | |||
github.com/tendermint/iavl v0.13.4-0.20200621145059-83c3470ad61d | |||
github.com/tendermint/iavl v0.13.4-0.20200622140716-a41a83e2415f |
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.
github.com/tendermint/iavl v0.13.4-0.20200622140716-a41a83e2415f | |
github.com/cosmos/iavl v0.13.4-0.20200622140716-a41a83e2415f |
Then go mod verify
and go mod tidy
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 go.mod in iavl has not migrated to cosmos/iavl
. We should do this in the next major release of IAVL
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.
go: github.com/cosmos/iavl@v0.13.4-0.20200622140716-a41a83e2415f: parsing go.mod:
module declares its path as: github.com/tendermint/iavl
but was required as: github.com/cosmos/iavl
Codecov Report
@@ Coverage Diff @@
## master #6492 +/- ##
=======================================
Coverage 56.34% 56.34%
=======================================
Files 470 470
Lines 28232 28232
=======================================
Hits 15907 15907
Misses 11200 11200
Partials 1125 1125 |
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.
ACK
Description
closes: #6486
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