Skip to content

Commit

Permalink
Disable wasm tests and add warning to README (#105)
Browse files Browse the repository at this point in the history
  • Loading branch information
tl-rodrigo-gryzinski committed Sep 7, 2023
1 parent c2d477e commit f1e71be
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 29 deletions.
29 changes: 0 additions & 29 deletions .github/workflows/ci.yml
Expand Up @@ -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
Expand Down
5 changes: 5 additions & 0 deletions README.md
Expand Up @@ -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
Expand Down

0 comments on commit f1e71be

Please sign in to comment.