Skip to content

Commit

Permalink
check wasm with default features
Browse files Browse the repository at this point in the history
As indicated in Azure#1554, we're not catching changes that impact real-world
WASM use in CICD.

Originally, our WASM implementation was a best-effort path as reqwest
(our first-class supported HTTP backend) did not support WASM.  Now that
reqwest supports WASM, we should update our CICD to validate WASM with
the default features.
  • Loading branch information
Brian Caswell committed Jan 8, 2024
1 parent f75406d commit 667d80e
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion eng/scripts/check_wasm.sh
Original file line number Diff line number Diff line change
Expand Up @@ -11,4 +11,4 @@ rustup update --no-self-update ${BUILD}
rustup target add --toolchain ${BUILD} wasm32-unknown-unknown

export RUSTFLAGS="-Dwarnings"
cargo +${BUILD} check --target=wasm32-unknown-unknown --no-default-features
cargo +${BUILD} check --target=wasm32-unknown-unknown

0 comments on commit 667d80e

Please sign in to comment.