Skip to content

Commit

Permalink
CI: Lint the crate using --all-targets --all-features
Browse files Browse the repository at this point in the history
We are not currently running the linter in CI, do so.
  • Loading branch information
tcharding committed Apr 1, 2024
1 parent f44943d commit 01a8642
Show file tree
Hide file tree
Showing 2 changed files with 10 additions and 0 deletions.
6 changes: 6 additions & 0 deletions .github/workflows/rust.yml
Original file line number Diff line number Diff line change
Expand Up @@ -15,6 +15,8 @@ jobs:
profile: minimal
toolchain: nightly
override: true
- name: Install clippy
run: rustup component add clippy
- name: Running benchmarks
env:
DO_BENCH: true
Expand All @@ -27,6 +29,10 @@ jobs:
env:
DO_FMT: true
run: ./contrib/test.sh
- name: Running linter
env:
DO_LINT: true
run: ./contrib/test.sh

Int-tests:
name: Integration tests
Expand Down
4 changes: 4 additions & 0 deletions contrib/test.sh
Original file line number Diff line number Diff line change
Expand Up @@ -33,6 +33,10 @@ fi
# Defaults / sanity checks
cargo test

if [ "$DO_LINT" = true ]; then
cargo +nightly clippy --all-targets --all-features -- -D warnings
fi

if [ "$DO_FEATURE_MATRIX" = true ]
then
# All features
Expand Down

0 comments on commit 01a8642

Please sign in to comment.