-
Notifications
You must be signed in to change notification settings - Fork 442
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
Add amino annotations to proto #1280
Conversation
You have successfully added a new CodeQL configuration |
@Mergifyio rebase |
Codecov Report
@@ Coverage Diff @@
## main #1280 +/- ##
==========================================
+ Coverage 63.44% 63.47% +0.02%
==========================================
Files 56 56
Lines 6971 6971
==========================================
+ Hits 4423 4425 +2
+ Misses 2219 2218 -1
+ Partials 329 328 -1
|
LGTM! |
* Add amino annotations to proto * Formatting only (cherry picked from commit 887ee0e) # Conflicts: # app/app.go # proto/cosmwasm/wasm/v1/proposal.proto # proto/cosmwasm/wasm/v1/tx.proto # x/wasm/keeper/msg_server_integration_test.go # x/wasm/types/proposal.pb.go # x/wasm/types/query.pb.go # x/wasm/types/tx.pb.go # x/wasm/types/types.pb.go
No backport as amino.proto was introduced > sdk 45 |
Resolves #1157
@pyramation please review: the failing tests/ linter/ simulations runs can be ignored as this stage. I am going to rebase the branch later. I am more interested if this PR includes all the annotations that you need