Skip to content

Commit

Permalink
Fix CI: ureq not supported on wasm
Browse files Browse the repository at this point in the history
ureq used to build on wasm, but apparently it never actually worked:
algesten/ureq#667

Now it no longer builds, so I've removed it from CI.
```
error: the wasm*-unknown-unknown targets are not supported by default, you may need to enable the "js" feature. For more information see: https://docs.rs/getrandom/#webassembly-support
Error:    --> /home/runner/.cargo/registry/src/index.crates.io-6f17d22bba15001f/getrandom-0.2.15/src/lib.rs:342:9
    |
342 | /         compile_error!("the wasm*-unknown-unknown targets are not supported by \
343 | |                         default, you may need to enable the \"js\" feature. \
344 | |                         For more information see: \
345 | |                         https://docs.rs/getrandom/#webassembly-support");
    | |________________________________________________________________________^

error[E0433]: failed to resolve: use of undeclared crate or module `imp`
Error:    --> /home/runner/.cargo/registry/src/index.crates.io-6f17d22bba15001f/getrandom-0.2.15/src/lib.rs:398:9
    |
398 |         imp::getrandom_inner(dest)?;
    |         ^^^ use of undeclared crate or module `imp`
```
  • Loading branch information
michaelkirk committed Aug 9, 2024
1 parent 1a83a0b commit 2e3ee72
Showing 1 changed file with 4 additions and 1 deletion.
5 changes: 4 additions & 1 deletion .github/workflows/test.yml
Original file line number Diff line number Diff line change
Expand Up @@ -9,6 +9,9 @@ jobs:
target:
- x86_64-unknown-linux-gnu
- wasm32-unknown-unknown
features:
- "reqwest-async,ureq-sync"
- "reqwest-async"

name: Build Check ${{ matrix.target }}
runs-on: ubuntu-22.04
Expand All @@ -28,7 +31,7 @@ jobs:
uses: actions-rs/cargo@v1
with:
command: check
args: --target ${{matrix.target}} --no-default-features --features=reqwest-async,ureq-sync
args: --target ${{matrix.target}} --no-default-features --features=${{matrix.features}}

test:
strategy:
Expand Down

0 comments on commit 2e3ee72

Please sign in to comment.