Skip to content

fix: update proto files to v0.34.21#547

Merged
tzdybal merged 1 commit intomainfrom
tzdybal/missing_proto
Oct 4, 2022
Merged

fix: update proto files to v0.34.21#547
tzdybal merged 1 commit intomainfrom
tzdybal/missing_proto

Conversation

@tzdybal
Copy link
Copy Markdown
Contributor

@tzdybal tzdybal commented Oct 3, 2022

During latest Cosmos SDK v0.46.1 update, I forgot to update proto files and script to get them. This PR fixes this.

@tzdybal tzdybal mentioned this pull request Oct 3, 2022
7 tasks
@tzdybal tzdybal self-assigned this Oct 3, 2022
Copy link
Copy Markdown
Contributor

@evan-forbes evan-forbes left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

do we need to regenerate anything? I'm guessing that the tests were failing before as they aren't related

@tzdybal
Copy link
Copy Markdown
Contributor Author

tzdybal commented Oct 3, 2022

Tests are failing because there is no compatible version of cosmos-sdk available. My plan was to rename to rollmint, and then release updated cosmos-sdk.

I run proto/gen.sh but there are no code changes visible - maybe we don't need this file rn at all? Lemme double check.

@tzdybal
Copy link
Copy Markdown
Contributor Author

tzdybal commented Oct 3, 2022

types.proto file is used by proto/optimint/optimint.proto and proto/optimint/state.proto, but changes in types.proto doesn't affect generated files (this time).

@tzdybal tzdybal requested a review from evan-forbes October 3, 2022 13:05
During latest Cosmos SDK v0.46.1 update, I forgot to update proto files
and script to get them. This PR fixes this.
@tzdybal tzdybal force-pushed the tzdybal/missing_proto branch from 1809126 to 702ee5c Compare October 4, 2022 12:36
@tzdybal tzdybal enabled auto-merge (squash) October 4, 2022 12:37
@tzdybal tzdybal merged commit 24dc3c4 into main Oct 4, 2022
@tzdybal tzdybal deleted the tzdybal/missing_proto branch October 4, 2022 12:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants