Skip to content

Commit f983578

Browse files
authored
ci(clang-tidy): run clang-tidy only for jazzy (#12433)
Remove the rosdistro matrix from clang-tidy-differential and clang-tidy-differential-cuda jobs, hardcoding jazzy. Signed-off-by: Mete Fatih Cırıt <mfc@autoware.org>
1 parent b1d8577 commit f983578

File tree

2 files changed

+3
-26
lines changed

2 files changed

+3
-26
lines changed

.github/workflows/build-test-tidy-pr.yaml

Lines changed: 0 additions & 24 deletions
Original file line numberDiff line numberDiff line change
@@ -111,41 +111,17 @@ jobs:
111111
needs:
112112
- check-if-cuda-job-is-needed
113113
- build-and-test-differential
114-
strategy:
115-
fail-fast: false
116-
matrix:
117-
rosdistro: [humble, jazzy]
118-
include:
119-
- rosdistro: humble
120-
container: ghcr.io/autowarefoundation/autoware:universe-devel
121-
- rosdistro: jazzy
122-
container: ghcr.io/autowarefoundation/autoware:universe-devel-jazzy
123-
124114
uses: ./.github/workflows/clang-tidy-differential.yaml
125115
with:
126-
rosdistro: ${{ matrix.rosdistro }}
127-
container: ${{ matrix.container }}
128116
run-condition: ${{ needs.check-if-cuda-job-is-needed.outputs.cuda_job_is_needed == 'false' && needs.build-and-test-differential.result == 'success' }}
129117

130118
clang-tidy-differential-cuda:
131119
if: ${{ always() }} # always run to provide report for status check
132120
needs:
133121
- check-if-cuda-job-is-needed
134122
- build-and-test-differential-cuda
135-
strategy:
136-
fail-fast: false
137-
matrix:
138-
rosdistro: [humble, jazzy]
139-
include:
140-
- rosdistro: humble
141-
container: ghcr.io/autowarefoundation/autoware:universe-devel
142-
- rosdistro: jazzy
143-
container: ghcr.io/autowarefoundation/autoware:universe-devel-jazzy
144-
145123
uses: ./.github/workflows/clang-tidy-differential.yaml
146124
with:
147-
rosdistro: ${{ matrix.rosdistro }}
148-
container: ${{ matrix.container }}
149125
container-suffix: -cuda
150126
run-condition: ${{ needs.check-if-cuda-job-is-needed.outputs.cuda_job_is_needed == 'true' && needs.build-and-test-differential-cuda.result == 'success' }}
151127
runner: "['self-hosted', 'Linux', 'X64']"

.github/workflows/clang-tidy-differential.yaml

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -4,15 +4,16 @@ on:
44
workflow_call:
55
inputs:
66
container:
7-
required: true
7+
required: false
8+
default: ghcr.io/autowarefoundation/autoware:universe-devel-jazzy
89
type: string
910
container-suffix:
1011
required: false
1112
default: ""
1213
type: string
1314
rosdistro:
1415
type: string
15-
default: humble
16+
default: jazzy
1617
required: false
1718
# Why is it a JSON string?
1819
# https://github.com/orgs/community/discussions/11692#discussioncomment-3541856

0 commit comments

Comments
 (0)