Skip to content

Commit 7ce6130

Browse files
committed
split clippy into a separate yaml for codeql
1 parent a1dfcbe commit 7ce6130

2 files changed

Lines changed: 43 additions & 29 deletions

File tree

.github/workflows/clippy.yml

Lines changed: 41 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,41 @@
1+
name: Clippy
2+
permissions:
3+
contents: read
4+
on:
5+
workflow_call:
6+
7+
jobs:
8+
rust-clippy-analyze:
9+
permissions:
10+
contents: read
11+
security-events: write
12+
if: (github.event_name != 'pull_request' && ! github.event.pull_request.head.repo.fork) || (github.event_name == 'pull_request' && github.event.pull_request.head.repo.fork)
13+
runs-on: ubuntu-latest
14+
steps:
15+
- name: Checkout code
16+
uses: actions/checkout@9c091bb21b7c1c1d1991bb908d89e4e9dddfe3e0 # v7.0.0
17+
with:
18+
submodules: recursive
19+
20+
- name: Install Rust toolchain
21+
uses: dtolnay/rust-toolchain@67ef31d5b988238dd797d409d6f9574278e20537 # v1.94.1
22+
with:
23+
toolchain: stable
24+
components: clippy
25+
26+
- name: Install required cargo
27+
run: cargo install clippy-sarif sarif-fmt
28+
29+
- name: Run rust-clippy
30+
run: cargo clippy
31+
--all-features
32+
--message-format=json | clippy-sarif | tee rust-clippy-results.sarif | sarif-fmt
33+
continue-on-error: true
34+
env:
35+
CARGO_TERM_VERBOSE: false
36+
37+
- name: Upload analysis results to GitHub
38+
uses: github/codeql-action/upload-sarif@8aad20d150bbac5944a9f9d289da16a4b0d87c1e # v4.36.2
39+
with:
40+
sarif_file: rust-clippy-results.sarif
41+
wait-for-processing: true

.github/workflows/rust.yml

Lines changed: 2 additions & 29 deletions
Original file line numberDiff line numberDiff line change
@@ -31,38 +31,11 @@ jobs:
3131
run: cargo fmt -- --check
3232

3333
rust-clippy-analyze:
34+
uses: ./.github/workflows/clippy.yml
3435
permissions:
36+
contents: read
3537
security-events: write
36-
if: (github.event_name != 'pull_request' && ! github.event.pull_request.head.repo.fork) || (github.event_name == 'pull_request' && github.event.pull_request.head.repo.fork)
37-
runs-on: ubuntu-latest
38-
steps:
39-
- name: Checkout code
40-
uses: actions/checkout@9c091bb21b7c1c1d1991bb908d89e4e9dddfe3e0 # v7.0.0
41-
with:
42-
submodules: recursive
4338

44-
- name: Install Rust toolchain
45-
uses: dtolnay/rust-toolchain@67ef31d5b988238dd797d409d6f9574278e20537 # v1.94.1
46-
with:
47-
toolchain: stable
48-
components: clippy
49-
50-
- name: Install required cargo
51-
run: cargo install clippy-sarif sarif-fmt
52-
53-
- name: Run rust-clippy
54-
run: cargo clippy
55-
--all-features
56-
--message-format=json | clippy-sarif | tee rust-clippy-results.sarif | sarif-fmt
57-
continue-on-error: true
58-
env:
59-
CARGO_TERM_VERBOSE: false
60-
61-
- name: Upload analysis results to GitHub
62-
uses: github/codeql-action/upload-sarif@8aad20d150bbac5944a9f9d289da16a4b0d87c1e # v4.36.2
63-
with:
64-
sarif_file: rust-clippy-results.sarif
65-
wait-for-processing: true
6639

6740
test-x86:
6841
if: (github.event_name != 'pull_request' && ! github.event.pull_request.head.repo.fork) || (github.event_name == 'pull_request' && github.event.pull_request.head.repo.fork)

0 commit comments

Comments
 (0)