0xPolygonMiden / miden-client

Client library that facilitates interaction with the Miden rollup
MIT License
32 stars 28 forks source link

fix: update build-wasm target from Makefile #419

Closed mFragaBA closed 3 months ago

mFragaBA commented 3 months ago

We missed a comment on #401 and currently the build for wasm is broken. This PR fixes that and introduces a build workflow to the CI that runs both make build and make build-wasm

igamigo commented 3 months ago

I think these changes look good, but we likely don't need to build the CLI as long as we have integration tests.

bobbinth commented 3 months ago

Should we also add badges to the main README for build jobs, similar to what we have in miden-base?

mFragaBA commented 3 months ago

Should we also add badges to the main README for build jobs, similar to what we have in miden-base?

good idea!