From f1e71bef3cd5cc8f26fba3fb544a6ed3aeeecaf3 Mon Sep 17 00:00:00 2001 From: tl-rodrigo-gryzinski <73602926+tl-rodrigo-gryzinski@users.noreply.github.com> Date: Thu, 7 Sep 2023 14:46:53 +0100 Subject: [PATCH] Disable wasm tests and add warning to README (#105) --- .github/workflows/ci.yml | 29 ----------------------------- README.md | 5 +++++ 2 files changed, 5 insertions(+), 29 deletions(-) diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml index bdd0217..60605ed 100644 --- a/.github/workflows/ci.yml +++ b/.github/workflows/ci.yml @@ -34,35 +34,6 @@ jobs: command: test args: --workspace --all-targets --features ${{ matrix.otel_version }} - wasm: - name: Compiles for the browser - runs-on: ubuntu-latest - strategy: - matrix: - otel_version: - - opentelemetry_0_13 - - opentelemetry_0_14 - - opentelemetry_0_15 - - opentelemetry_0_16 - - opentelemetry_0_17 - - opentelemetry_0_18 - - opentelemetry_0_19 - - opentelemetry_0_20 - steps: - - name: Checkout repository - uses: actions/checkout@v2 - - name: Install Rust - uses: actions-rs/toolchain@v1 - with: - target: wasm32-unknown-unknown - toolchain: stable - profile: minimal - override: true - - uses: actions-rs/cargo@v1 - with: - command: build - args: --workspace --features ${{ matrix.otel_version }} --target wasm32-unknown-unknown - rustfmt: name: Rustfmt runs-on: ubuntu-latest diff --git a/README.md b/README.md index 0890ca8..dddf05d 100644 --- a/README.md +++ b/README.md @@ -15,6 +15,11 @@ implementations. This repository also contains a couple of useful concrete middl * [`reqwest-tracing`](https://crates.io/crates/reqwest-tracing): [`tracing`](https://crates.io/crates/tracing) integration, optional opentelemetry support. +Note about browser support: automated tests targetting wasm are disabled. The crate may work with +wasm but wasm support is unmaintained. PRs improving wasm are still welcome but you'd need to +reintroduce the tests and get them passing before we'd merge it (see +https://github.com/TrueLayer/reqwest-middleware/pull/105). + ## Overview The `reqwest-middleware` client exposes the same interface as a plain `reqwest` client, but