Skip to content
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

Merged
merged 2 commits into from
Mar 31, 2023
Merged

Add amino annotations to proto #1280

merged 2 commits into from
Mar 31, 2023

Conversation

alpe
Copy link
Contributor

@alpe alpe commented Mar 21, 2023

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

@github-advanced-security
Copy link

You have successfully added a new CodeQL configuration .github/workflows/codeql-analizer.yml:CodeQL-Build. As part of the setup process, we have scanned this repository and found 12 existing alerts. Please check the repository Security tab to see all alerts.

@alpe alpe requested a review from pyramation March 22, 2023 09:41
@alpe alpe added the blocked label Mar 24, 2023
@alpe
Copy link
Contributor Author

alpe commented Mar 29, 2023

@Mergifyio rebase

@alpe alpe changed the base branch from deprecate_proposal_types to main March 30, 2023 08:31
@codecov
Copy link

codecov bot commented Mar 30, 2023

Codecov Report

Merging #1280 (8effbc9) into main (674eceb) will increase coverage by 0.02%.
The diff coverage is 100.00%.

Impacted file tree graph

@@            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     
Impacted Files Coverage Δ
app/app.go 86.27% <ø> (ø)
x/wasm/types/codec.go 100.00% <100.00%> (ø)

... and 1 file with indirect coverage changes

@alpe alpe removed the blocked label Mar 30, 2023
@pyramation
Copy link
Contributor

LGTM!

@alpe alpe merged commit 887ee0e into main Mar 31, 2023
@alpe alpe deleted the 1157_annotations branch March 31, 2023 06:29
@alpe alpe added the backport/v0.3x Backport patches to sdk45 release branch label May 11, 2023
mergify bot pushed a commit that referenced this pull request May 11, 2023
* 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
@alpe
Copy link
Contributor Author

alpe commented May 11, 2023

No backport as amino.proto was introduced > sdk 45

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport/v0.3x Backport patches to sdk45 release branch
Projects
None yet
Development

Successfully merging this pull request may close these issues.

new proto annotations
2 participants