Skip to content

Commit 196e9d2

Browse files
authored
ci: Run Clippy on all platforms (#452)
2 parents 5d175d9 + f26bc17 commit 196e9d2

File tree

1 file changed

+5
-5
lines changed

1 file changed

+5
-5
lines changed

.github/workflows/tests.yml

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -59,7 +59,6 @@ jobs:
5959
run: |
6060
cargo --version
6161
rustc --version
62-
# TODO: Maybe also check clippy and rustfmt here.
6362
- name: Build
6463
run: cargo build --all-targets
6564
- uses: taiki-e/install-action@v2
@@ -71,8 +70,7 @@ jobs:
7170
- name: Check rustfmt
7271
run: cargo fmt --all --check
7372
- name: Check clippy
74-
# TODO: -- -D warnings
75-
run: cargo clippy --all-targets --all-features
73+
run: cargo clippy --all-targets --all-features -- -D warnings
7674
- name: Check typos
7775
uses: crate-ci/typos@master
7876
- name: Build release binary
@@ -115,8 +113,10 @@ jobs:
115113
- name: Check rustfmt
116114
run: cargo fmt --all --check
117115
- name: Check clippy
118-
# TODO: Deny warnings
119-
run: cargo clippy --all-targets --all-features
116+
if: matrix.version == 'stable'
117+
# Clippy checks can vary between versions in a way that makes it a bit
118+
# fiddly to satisfy them all, so only insist that they pass on stable.
119+
run: cargo clippy --all-targets --all-features -- -D warnings
120120
- run: cargo update
121121
- name: Test after cargo update
122122
run: cargo test --workspace

0 commit comments

Comments
 (0)