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

Disable wasm build in build CI #3083

Merged
merged 1 commit into from
Aug 9, 2022
Merged

Disable wasm build in build CI #3083

merged 1 commit into from
Aug 9, 2022

Conversation

epilys
Copy link
Contributor

@epilys epilys commented Aug 9, 2022

Wasm build was added unnecessarily in the build github action in
4710a2a

The wasm build is not currently part of the binary assets of a release,
so this commits disable it.

By the way, the wasm build fails because the wasm32-wasi target is
not added with rustup before the make build-wasmer-wasm is called.

Wasm build was added unnecessarily in the build github action in
4710a2a

The wasm build is not currently part of the binary assets of a release,
so this commits disable it.

By the way, the wasm build fails because the wasm32-wasi target is
not added with rustup before the `make build-wasmer-wasm` is called.
@epilys epilys self-assigned this Aug 9, 2022
@epilys
Copy link
Contributor Author

epilys commented Aug 9, 2022

bors r+

@ptitSeb
Copy link
Contributor

ptitSeb commented Aug 9, 2022

wasm build was added to avoid it being broken by code change

@bors
Copy link
Contributor

bors bot commented Aug 9, 2022

Build succeeded:

@bors bors bot merged commit a6ddf4e into master Aug 9, 2022
@bors bors bot deleted the fix-ci-failure branch August 9, 2022 12:36
@epilys
Copy link
Contributor Author

epilys commented Aug 9, 2022

wasm build was added to avoid it being broken by code change

Makes sense, maybe it'd be better to put it in the test workflows.

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