Skip to content

Commit

Permalink
Omit --tests argument from cargo clippy CI job
Browse files Browse the repository at this point in the history
The --all-targets argument that we provide to the cargo clippy
invocation already captures all targets. As such, there is no reason to
specify --tests as well. Remove this argument.
  • Loading branch information
d-e-s-o committed Jun 15, 2024
1 parent fa6c36c commit d968873
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions .github/workflows/test.yml
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
# Copyright (C) 2022-2023 Daniel Mueller <deso@posteo.net>
# Copyright (C) 2022-2024 Daniel Mueller <deso@posteo.net>
# SPDX-License-Identifier: GPL-3.0-or-later

name: Test
Expand Down Expand Up @@ -64,7 +64,7 @@ jobs:
- uses: actions/checkout@v3
- uses: dtolnay/rust-toolchain@stable
- uses: Swatinem/rust-cache@v2
- run: cargo clippy --no-deps --all-targets --all-features --tests -- -A unknown_lints -A deprecated -D warnings
- run: cargo clippy --no-deps --all-targets --all-features -- -A unknown_lints -A deprecated -D warnings
rustfmt:
name: Check code formatting
runs-on: ubuntu-latest
Expand Down

0 comments on commit d968873

Please sign in to comment.