Skip to content

Commit

Permalink
Merge pull request #764 from pohlm01/main
Browse files Browse the repository at this point in the history
Fail CI on Clippy warnings
  • Loading branch information
rnijveld authored Sep 19, 2023
2 parents fbfbabd + dc17cac commit 45e841d
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions .github/workflows/ci.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -171,11 +171,11 @@ jobs:
- name: clippy sudo-test
working-directory: test-framework
run: cargo clippy -p sudo-test --no-deps -- --deny clippy::all
run: cargo clippy -p sudo-test --no-deps -- --deny warnings

- name: clippy compliance-tests
working-directory: test-framework
run: cargo clippy -p sudo-compliance-tests --tests --no-deps -- --deny clippy::all
run: cargo clippy -p sudo-compliance-tests --tests --no-deps -- --deny warnings

- name: Check that all ignored tests are linked to a GH issue
working-directory: test-framework/sudo-compliance-tests
Expand Down Expand Up @@ -401,7 +401,7 @@ jobs:
with:
name: clippy-result
token: ${{ secrets.GITHUB_TOKEN }}
args: --no-deps
args: --no-deps -- --deny warnings

docs:
needs: clippy
Expand Down

0 comments on commit 45e841d

Please sign in to comment.