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

feat: Support for wasmd >=0.29 by generating files via buf #332

Closed
wants to merge 2 commits into from

Conversation

apollo-sturdy
Copy link
Contributor

@apollo-sturdy apollo-sturdy commented Feb 24, 2023

Since wasmd >= 0.29 started using buf, it broke proto-build. This PR fixes it by instead generating the files via buf. Needless to say this changes requires users to have buf installed. See here for more context: #318 (comment)

@tony-iqlusion
Copy link
Member

@apollo-sturdy can you rebase? that should take care of the MSRV issues

@apollo-sturdy apollo-sturdy force-pushed the dev/generate-via-buf branch from 38010da to 5115198 Compare March 7, 2023 15:28
@apollo-sturdy
Copy link
Contributor Author

@apollo-sturdy can you rebase? that should take care of the MSRV issues

Done!

@tony-iqlusion
Copy link
Member

Looks like cosmrs needs to be updated due to the schema changes.

@apollo-sturdy not sure if you want to take that on or not. If not, I can

@apollo-sturdy
Copy link
Contributor Author

Looks like cosmrs needs to be updated due to the schema changes.

@apollo-sturdy not sure if you want to take that on or not. If not, I can

Would be great if you could do it. I'm not too familiar with it or the main Cosmos SDK types as I mostly work with CosmWasm.

@tony-iqlusion
Copy link
Member

Merged in #358

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants