Skip to content

Commit

Permalink
ci: Run Clippy on all platforms (#452)
Browse files Browse the repository at this point in the history
  • Loading branch information
sourcefrog authored Dec 8, 2024
2 parents 5d175d9 + f26bc17 commit 196e9d2
Showing 1 changed file with 5 additions and 5 deletions.
10 changes: 5 additions & 5 deletions .github/workflows/tests.yml
Original file line number Diff line number Diff line change
Expand Up @@ -59,7 +59,6 @@ jobs:
run: |
cargo --version
rustc --version
# TODO: Maybe also check clippy and rustfmt here.
- name: Build
run: cargo build --all-targets
- uses: taiki-e/install-action@v2
Expand All @@ -71,8 +70,7 @@ jobs:
- name: Check rustfmt
run: cargo fmt --all --check
- name: Check clippy
# TODO: -- -D warnings
run: cargo clippy --all-targets --all-features
run: cargo clippy --all-targets --all-features -- -D warnings
- name: Check typos
uses: crate-ci/typos@master
- name: Build release binary
Expand Down Expand Up @@ -115,8 +113,10 @@ jobs:
- name: Check rustfmt
run: cargo fmt --all --check
- name: Check clippy
# TODO: Deny warnings
run: cargo clippy --all-targets --all-features
if: matrix.version == 'stable'
# Clippy checks can vary between versions in a way that makes it a bit
# fiddly to satisfy them all, so only insist that they pass on stable.
run: cargo clippy --all-targets --all-features -- -D warnings
- run: cargo update
- name: Test after cargo update
run: cargo test --workspace
Expand Down

0 comments on commit 196e9d2

Please sign in to comment.