-
Notifications
You must be signed in to change notification settings - Fork 332
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 proto-compiler as a proper CLI app #365
Conversation
ef73079
to
da8c188
Compare
da8c188
to
24ef33d
Compare
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.
Looks great! Thanks Romain!
Great UX :) |
Actually, the cosmos-sdk for stargate-4 is at |
Agreed! Let's do that in a follow-up PR so that we unblock #361 already. |
* Refactor proto-compiler into a proper CLI app * Allow specifying the commit to checkout the SDK at * Do a proper checkout of the given commit * Update README * Emit Cosmos SDK version (commit hash) to a file and expose it in the lib * Change SDK version to <unknown> for now * Update changelog
Closes: cosmos/ibc-proto-rs#17
Description
proto-compiler
crate into a proper CLI appproto
library as a constant.ibc-proto-compiler
ibc-proto-compiler clone-sdk
ibc-proto-compiler compile
For contributor use:
docs/
) and code comments.Files changed
in the Github PR explorer.