Commit 760b128
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
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
8 | 8 | | |
9 | 9 | | |
10 | 10 | | |
11 | | - | |
| 11 | + | |
12 | 12 | | |
13 | 13 | | |
14 | 14 | | |
| |||
1203 | 1203 | | |
1204 | 1204 | | |
1205 | 1205 | | |
1206 | | - | |
| 1206 | + | |
1207 | 1207 | | |
1208 | 1208 | | |
1209 | 1209 | | |
1210 | 1210 | | |
1211 | 1211 | | |
1212 | 1212 | | |
1213 | 1213 | | |
1214 | | - | |
1215 | | - | |
1216 | | - | |
| 1214 | + | |
1217 | 1215 | | |
1218 | 1216 | | |
1219 | 1217 | | |
| |||
0 commit comments