From 121d05ca56af8c7ccf2b89bb2a9a6984ab36edef Mon Sep 17 00:00:00 2001 From: Harald Hoyer Date: Mon, 12 Jan 2026 08:58:34 +0100 Subject: [PATCH] fix: add --all-features to coverage and lint workflows MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit The coverage and clippy workflows were not testing async code because they ran without --all-features. The tokio feature enables async support in the varlink crate, so without it: - Code coverage excluded all async code paths - Clippy did not lint async code The CI workflow already had --all-features on line 47, so tests were running correctly, but coverage and linting were incomplete. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude --- .github/workflows/coverage.yml | 2 +- .github/workflows/lint.yml | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/.github/workflows/coverage.yml b/.github/workflows/coverage.yml index 2364dff..a07ee7c 100644 --- a/.github/workflows/coverage.yml +++ b/.github/workflows/coverage.yml @@ -27,7 +27,7 @@ jobs: - name: Run cargo-llvm-cov run: | mkdir coverage - cargo llvm-cov --workspace --lcov --output-path ./coverage/lcov.info + cargo llvm-cov --workspace --all-features --lcov --output-path ./coverage/lcov.info - name: Upload to coveralls uses: coverallsapp/github-action@master diff --git a/.github/workflows/lint.yml b/.github/workflows/lint.yml index bd8dd9f..d0c6a28 100644 --- a/.github/workflows/lint.yml +++ b/.github/workflows/lint.yml @@ -36,4 +36,4 @@ jobs: - uses: actions-rs/cargo@v1 with: command: clippy - args: -- -D warnings + args: --all-features -- -D warnings