Skip to content

Commit f185fae

Browse files
committed
ci(health-check-pr): always run, gate work via changed-files
Required status checks were blocked when the workflow's `paths:` filter excluded a PR: the check never appeared, so branch protection waited forever. Drop `paths:` and gate `require-label` (and transitively `health-check`) on `step-security/changed-files`. Skipped jobs satisfy required checks; missing workflows do not. Signed-off-by: Mete Fatih Cırıt <mfc@autoware.org>
1 parent e0baba8 commit f185fae

1 file changed

Lines changed: 23 additions & 8 deletions

File tree

.github/workflows/health-check-pr.yaml

Lines changed: 23 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -7,21 +7,36 @@ on:
77
- synchronize
88
- reopened
99
- labeled
10-
paths:
11-
- repositories/*.repos
12-
- .github/workflows/health-check.yaml
13-
- .github/workflows/health-check-pr.yaml
14-
- .github/workflows/health-check-reusable.yaml
15-
- ansible-galaxy-requirements.yaml
16-
- ansible/**
17-
- docker-new/**
1810

1911
concurrency:
2012
group: ${{ github.workflow }}-${{ github.ref }}
2113
cancel-in-progress: true
2214

2315
jobs:
16+
changed-files:
17+
runs-on: ubuntu-latest
18+
outputs:
19+
should-run: ${{ steps.check.outputs.any_changed }}
20+
steps:
21+
- uses: actions/checkout@v6
22+
with:
23+
fetch-depth: 0
24+
25+
- id: check
26+
uses: step-security/changed-files@v47
27+
with:
28+
files: |
29+
repositories/*.repos
30+
.github/workflows/health-check.yaml
31+
.github/workflows/health-check-pr.yaml
32+
.github/workflows/health-check-reusable.yaml
33+
ansible-galaxy-requirements.yaml
34+
ansible/**
35+
docker-new/**
36+
2437
require-label:
38+
needs: changed-files
39+
if: needs.changed-files.outputs.should-run == 'true'
2540
uses: autowarefoundation/autoware-github-actions/.github/workflows/require-label.yaml@v1
2641
with:
2742
label: run:health-check

0 commit comments

Comments
 (0)