-
Notifications
You must be signed in to change notification settings - Fork 94
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
ci(ibc-clients): upload Tendermint CosmWasm client as workflow artifact #1173
Conversation
cancel-in-progress: true | ||
|
||
jobs: | ||
upload-tm-wasm08-client: |
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.
upload-tm-wasm08-client: | |
upload-tendermint-cw-client: |
This should be adjusted for other places where wasm08
used. Actually, the 08-wasm
is a distinct type of light client that enables operating the CosmWasm contract of other clients like in our case the CosmWasm contract of the Tendermint.
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.
Thanks for adding this. Left just a small comment.
…ct (#1173) * tendermint wasm08 client as workflow artifact * rm retention days * lints on comments * conditional retention days * use actions-rust-lang over dtolnay * imp docker run * use cosmwasm/optimizer 0.15.1 * correct mount paths * validate checksum * s/wasm08/cosmwasm * .yaml over .yml * install cargo binary before wasm compile to avoid docker mount permission * fix permissions manually * use id * tendermint over tm * document tendermint cw artifact * grammar
Closes: #1170
Example workflow run: 8738022488.
cosmwasm/optimizer
.cosmwasm-check
on the compiled cw contract.Also, contains an example job to download the artifact at remote repository.
References:
Description
PR author checklist:
unclog
.docs/
).Reviewer checklist:
Files changed
in the GitHub PR explorer.