Skip to content

Commit

Permalink
ci: run clippy w/ --all-targets
Browse files Browse the repository at this point in the history
This ensures test code is also linted.
  • Loading branch information
cpu committed Apr 19, 2024
1 parent aac37ad commit 3dc23ef
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions .github/workflows/test.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -181,7 +181,7 @@ jobs:
# (below) will have a new lint that we want to suppress.
# If we suppress (e.g. #![allow(clippy::arc_with_non_send_sync)]),
# we would get an unknown-lint error from older clippy versions.
run: cargo clippy --locked --workspace -- -D warnings -A unknown-lints
run: cargo clippy --locked --workspace --all-targets -- -D warnings -A unknown-lints

clippy-nightly-optional:
name: Clippy nightly (optional)
Expand All @@ -196,7 +196,7 @@ jobs:
with:
components: clippy
- name: Check clippy
run: cargo clippy --locked --workspace -- -D warnings
run: cargo clippy --locked --workspace --all-targets -- -D warnings

clang-tidy:
name: Clang Tidy
Expand Down

0 comments on commit 3dc23ef

Please sign in to comment.