-
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
refactor(codec)!: update codec to x/tx v0.6.0 #15873
Merged
Merged
Changes from all commits
Commits
Show all changes
50 commits
Select commit
Hold shift + click to select a range
0331b97
fix for 32 bit platforms
kocubinski 3310120
use interface for protoregistry.Files struct
kocubinski 78247ad
Add default mode
kocubinski 2d9bc1b
refactor(codec)!: update codec to x/tx v0.6.0
aaronc e997676
fix tests, add signing.Context.Validate
aaronc 1bd5544
update go.mod
aaronc a7cc17a
CHANGELOG.md
aaronc 0b531ed
add validation check
aaronc 0dc96b8
go mod tidy
aaronc 68273ac
Merge branch 'main' of github.com:cosmos/cosmos-sdk into aaronc/11275…
kocubinski 93475ea
fix decode tests
kocubinski 059e1ac
simpler
kocubinski a8628b8
try to fix validate
aaronc 97aaef5
more hacky address codec fix
kocubinski 7d83030
fix validate another way
aaronc 2de4eed
Merge remote-tracking branch 'origin/aaronc/11275-update-codec' into …
aaronc a4ad892
Merge branch 'main' of github.com:cosmos/cosmos-sdk into aaronc/11275…
aaronc ed17c8c
merge
aaronc bd2b706
Merge branch 'main' of github.com:cosmos/cosmos-sdk into aaronc/11275…
kocubinski c0f4eda
remove unused code
kocubinski e0a3553
Merge branch 'main' of github.com:cosmos/cosmos-sdk into kocubinski/x…
kocubinski 2128b1a
Merge branch 'kocubinski/x-tx-0.5.x' into aaronc/11275-update-codec
kocubinski c9c34ed
Merge branch 'main' of github.com:cosmos/cosmos-sdk into aaronc/11275…
kocubinski f899dcf
Merge branch 'main' of github.com:cosmos/cosmos-sdk into aaronc/11275…
kocubinski 5bd54fe
use x/tx v0.6.1
kocubinski 4c34673
go mod tidy
kocubinski 1e5b778
Squashed commit of the following:
kocubinski 0439089
fix some errors
kocubinski b45e0bb
sensible refactor and cleanup for NewTxConfig
kocubinski f88ca77
delete unused code
kocubinski d568178
lint fix and rosetta go mod tidy
kocubinski 8a00f65
Merge branch 'main' into aaronc/11275-update-codec
kocubinski 9c76023
add comments
kocubinski 409ed10
linting fixes
kocubinski 76f5774
fix rosetta tests
kocubinski 2dc5a5a
Merge branch 'main' of github.com:cosmos/cosmos-sdk into aaronc/11275…
kocubinski c8c4893
gofumpt
kocubinski 49541c4
clean up tx.ConfigOptions a bit
kocubinski e623044
fix usage
kocubinski 2c71536
nit
kocubinski bf0830d
move variadic arg to options struct
kocubinski fe9f831
fix usage
kocubinski cae22dd
go mod tidy
kocubinski 23398ce
Merge branch 'main' of github.com:cosmos/cosmos-sdk into aaronc/11275…
kocubinski 1711502
fix failing simapp_v1 test
kocubinski ea46ef4
Merge branch 'main' of github.com:cosmos/cosmos-sdk into aaronc/11275…
kocubinski f92230d
rosetta go.mod fix after merge
kocubinski 5dae4ed
address review comments
aaronc 8211a3a
Merge branch 'main' of github.com:cosmos/cosmos-sdk into aaronc/11275…
aaronc 506ecf9
remove unused import
kocubinski 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
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.
Does changing this interface contract require a mention in the CHANGELOG?
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.
No, it's new anyway and there are CHANGELOG entries already for this addition