-
Notifications
You must be signed in to change notification settings - Fork 625
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
ibc-go v5: cosmos-sdk 46 #1653
Merged
Merged
ibc-go v5: cosmos-sdk 46 #1653
Changes from 74 commits
Commits
Show all changes
88 commits
Select commit
Hold shift + click to select a range
800d530
begin changes needed for "real 46"
faddat 0f64e06
46 updates
faddat 8867120
46 updates
faddat 30f7946
Delete .gitpod.yml
faddat 0c77eab
46 updates
faddat 1e79e0b
46 updates
faddat 185adee
Update genesis_test.go
faddat 11d96cb
update tests
faddat bdf7cc3
Merge branch 'real-46-branch' of https://github.com/notional-labs/ibc…
faddat ab06157
bring back the client client
faddat 6704daf
fumpt
faddat 42c1eac
genesis with val set
vuong177 78689e1
fix TestRandomizedGenState
nghuyenthevinh2000 32fa306
Merge branch 'main' into real-46-branch
faddat e300204
Merge branch 'main' into real-46-branch
faddat 8e4f87e
use cosmos-sdk v0.46.0-rc2
faddat 4b579fa
Merge branch 'main' into real-46-branch
faddat 93cd207
Revert "Merge branch 'main' into real-46-branch"
faddat 11e336c
Merge remote-tracking branch 'upstream/main' into real-46-branch
faddat b0b34cf
goimports
faddat ee64d56
fumpt
faddat 06b8deb
Update testing/simapp/app.go
faddat 8c2c541
Merge branch 'main' into real-46-branch
faddat df546ed
Update testing/simapp/simd/cmd/genaccounts.go
faddat eff83e2
make format
faddat 626480f
tidy
faddat 496a7f3
change import path
faddat d6ee5c9
remove gitpod artifact
faddat 1f290e0
sdk.Int -> math.Int
faddat cf0997e
group sdk and tendermint imports
faddat 869043d
Organize Governance Proposal Handlers
faddat 802b1bc
Remove Extra line in msgs_test.go
faddat 232cbb6
use test.yml from main branch
faddat 1281e74
fix fee module ibc middleware
faddat 8769f22
remove sdk_test.go and use Carlos' branch's antehandler
faddat 2caf65f
various fixes
faddat 8dc9b01
fix assignment to addr
faddat 9f95dd5
revert change in misbehaviour_handle.go
vuong177 0c3b31f
Merge branch 'main' into real-46-branch
faddat 6a063fe
e2e go.mod
faddat 21cf683
e2e v4 module path
faddat 55b166b
Merge branch 'main' into real-46-branch
faddat 4bd8fdb
Merge branch 'main' into real-46-branch
faddat 859eae7
Merge branch 'main' into real-46-branch
faddat 374e043
fix import paths
faddat 33be3d6
Merge branch 'main' into real-46-branch
faddat 9441c40
Merge branch 'main' into real-46-branch
faddat d0ae698
goimports and gofumpt
faddat eb12e81
Merge branch 'main' into real-46-branch
faddat 8afece8
Merge branch 'main' into real-46-branch
faddat 4951539
Merge branch 'main' into real-46-branch
faddat 2233d7b
46 final release
faddat a2d6e01
fix makefile
faddat e09af23
update proto file to v5
vuong177 0eb3fc0
Merge remote-tracking branch 'upstream/main' into real-46-branch
faddat 771e422
Merge remote-tracking branch 'upstream/main' into real-46-branch
faddat 0a11156
temp gitpod for protos
faddat 8f3943d
work on protos
faddat 92061fc
Merge remote-tracking branch 'upstream/main' into real-46-branch
faddat a6e2ce3
tidy
faddat 272b708
Delete .gitpod.yml
faddat 6f2e6df
Merge branch 'main' into real-46-branch
faddat cf43ee0
Merge remote-tracking branch 'upstream/main' into real-46-branch
faddat 527d656
finish merge
faddat cbebf62
Merge branch 'main' into real-46-branch
faddat 89e53d2
apply review suggestions, fix imports
colin-axner 76ee58d
remove incorrect e2e import path
colin-axner 2201809
fix incorrect import moving
colin-axner c4ca865
Merge pull request #75 from cosmos/colin/review-suggestions
faddat 040aca9
Merge branch 'main' into real-46-branch
faddat acff013
refactor acknowledgement construction for SDK 0.46
crodriguezvega ad9241a
Merge branch 'main' into real-46-branch
catShaark e31ec28
Merge pull request #76 from crodriguezvega/carlos/refactor-acknowledg…
faddat db99001
46 upgrade
faddat 148c020
fix ics27 changes, import fixes
colin-axner 9bc1af7
Merge branch 'main' into real-46-branch
faddat d0b020b
Merge pull request #77 from cosmos/colin/self-review
faddat 993ce8b
re add ibcclient proposal route
catShaark f300949
Merge branch 'main' into real-46-branch
catShaark d36576c
Merge branch 'main' into real-46-branch
catShaark 053141f
fix for make proto-all
crodriguezvega 7728fa1
Merge branch 'main' into real-46-branch
faddat 70ee541
Merge pull request #79 from crodriguezvega/carlos/proto-fix
faddat d48007f
now both commands work, together!
faddat 08f60a0
tidy
faddat 171a81e
update swagger docs
faddat d11a462
appease codeql
faddat 9af421e
update CONFIO_URL
faddat File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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.
Is there release version of
confio/ics23that we can use here?I think it would best to avoid using a commit hash
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.
cc @ethanfrey
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.
Looks like 0.7.1 is the latest stable tag https://github.com/confio/ics23/tags
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.
sadly, it didn't work -- the correct release though, is v0.7.1
(no clue why it wouldn't work)