Skip to content

Commit 2ceba8d

Browse files
committed
ci: also run clippy on rmcp with all features except local
--all-features enables the local feature, which cfg-gates out tower.rs and most of the test suite, so the existing clippy step never actually lints them.
1 parent b0d3b8b commit 2ceba8d

1 file changed

Lines changed: 9 additions & 1 deletion

File tree

.github/workflows/ci.yml

Lines changed: 9 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -61,9 +61,17 @@ jobs:
6161

6262
- uses: Swatinem/rust-cache@6323deb102c322ba6fcbdcafc7e3dddab59af2b6 # v2
6363

64-
- name: Run clippy
64+
- name: Run clippy (all features)
6565
run: cargo clippy --all-targets --all-features -- -D warnings
6666

67+
- name: Run clippy (all features except local)
68+
run: |
69+
FEATURES=$(cargo metadata --no-deps --format-version 1 \
70+
| jq -r '[.packages[] | select(.name == "rmcp") | .features | keys[]
71+
| select(startswith("__") | not)
72+
| select(. != "local")] | join(",")')
73+
cargo clippy --package rmcp --all-targets --no-default-features --features "$FEATURES" -- -D warnings
74+
6775
semver:
6876
name: SemVer Check
6977
runs-on: ubuntu-latest

0 commit comments

Comments
 (0)