Skip to content

Commit 760b128

Browse files
committed
fix(lighthouse-ci): run on every PR + nightly, drop label gate
Two changes that simplify the trigger surface and fix the HIGH-severity review-bot finding on PR #20850 about the workflow-level `labeled` trigger: 1. Remove `labeled` from `pull_request.types` in build.yml. With it included, adding ANY label to a PR (not just `ci:lighthouse`) re-triggered the whole workflow, and the workflow-level `concurrency.cancel-in-progress: true` block cancelled the in-progress build/test/e2e jobs and restarted them from scratch. Labels are no longer a CI control surface here. 2. Drop the `contains(labels, 'ci:lighthouse')` check on `job_lighthouse_matrix`. Lighthouse now runs on every PR push the same way the other test jobs do, plus on the nightly schedule. Simpler mental model, no per-PR opt-in. The Lighthouse jobs are still NOT referenced in `job_required_jobs_passed.needs`, so they never block merges \u2014 only surface measurements via the PR comment and (after the next commit) the workflow run Job Summary.
1 parent 1549ea8 commit 760b128

1 file changed

Lines changed: 3 additions & 5 deletions

File tree

.github/workflows/build.yml

Lines changed: 3 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,7 @@ on:
88
- v8
99
- release/**
1010
pull_request:
11-
types: [opened, synchronize, reopened, labeled]
11+
types: [opened, synchronize, reopened]
1212
merge_group:
1313
types: [checks_requested]
1414
workflow_dispatch:
@@ -1203,17 +1203,15 @@ jobs:
12031203
if-no-files-found: ignore
12041204

12051205
# -----------------------------------------------------------------------
1206-
# Lighthouse CI — nightly + label-gated (ci:lighthouse)
1206+
# Lighthouse CI — runs on every PR and on the nightly schedule.
12071207
# NOT in job_required_jobs_passed — never blocks merges.
12081208
# -----------------------------------------------------------------------
12091209

12101210
job_lighthouse_matrix:
12111211
name: Lighthouse Matrix
12121212
needs: [job_get_metadata]
12131213
runs-on: ubuntu-24.04
1214-
if: |
1215-
github.event_name == 'schedule' ||
1216-
(github.event_name == 'pull_request' && contains(github.event.pull_request.labels.*.name, 'ci:lighthouse'))
1214+
if: github.event_name == 'pull_request' || github.event_name == 'schedule'
12171215
outputs:
12181216
matrix: ${{ steps.matrix.outputs.matrix }}
12191217
steps:

0 commit comments

Comments
 (0)