Skip to content

fleet-status: run on a label event only for the verified label #200

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
askalf merged 1 commit into from
Nov 2, 2026
Merged

fleet-status: run on a label event only for the verified label #200

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion .github/workflows/fleet-status-backfill.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -21,6 +21,6 @@ jobs:
checks: read
statuses: write
steps:
- uses: askalf/ci/actions/fleet-status@4365d12f956b60e795b5b55451e2035549a3600c # main 2026-11-01, askalf/ci#153
- uses: askalf/ci/actions/fleet-status@84d68a04c25cbc7c50598b05476293c4ae00a898 # main 2026-11-02, askalf/ci#159
with:
all-open: 'true'
2 changes: 1 addition & 1 deletion .github/workflows/fleet-status-self-test.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -28,6 +28,6 @@ jobs:
with:
persist-credentials: false

- uses: askalf/ci/actions/fleet-status-check@cf3327ecdb3de75efa601181bd5ca91c2e17ec2a # main 2026-11-02, askalf/ci#157
- uses: askalf/ci/actions/fleet-status-check@84d68a04c25cbc7c50598b05476293c4ae00a898 # main 2026-11-02, askalf/ci#159
with:
min-own-exprs: '5'
8 changes: 6 additions & 2 deletions .github/workflows/fleet-status.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -45,6 +45,8 @@ jobs:
# issue_comment fires for issues too; only PR comments matter. The script re-checks it.
# A comment starts a run only when it can change a status: a verification comment, an edit
# away from one, or its deletion.
# Of the labels, the lanes read only `verified`, so a label added or removed runs the job only
# when it is that one (a Dependabot PR adds two others as it opens).
if: >-
(github.event_name == 'issue_comment' && github.event.issue.pull_request != null &&
(startsWith(github.event.comment.body, '## Verification at') ||
Expand All @@ -56,7 +58,9 @@ jobs:
(contains(fromJSON('["pull_request","pull_request_target","pull_request_review"]'), github.event.workflow_run.event) &&
github.event.workflow_run.head_repository.full_name != github.repository))) ||
(github.event_name != 'issue_comment' && github.event_name != 'workflow_run' &&
github.event.pull_request.head.repo.full_name == github.repository)
github.event.pull_request.head.repo.full_name == github.repository &&
(!contains(fromJSON('["labeled","unlabeled"]'), github.event.action) ||
github.event.label.name == 'verified'))
# On our runner for every event, a fork's included: the job runs the pinned fleet-status action
# and never PR code (see above), so GitHub's hosted queue going down does not hold the fleet/verify
# and fleet/review statuses a merge waits on. The runner is this repository's fleet status row
Expand All @@ -71,7 +75,7 @@ jobs:
checks: read
statuses: write
steps:
- uses: askalf/ci/actions/fleet-status@4365d12f956b60e795b5b55451e2035549a3600c # main 2026-11-01, askalf/ci#153
- uses: askalf/ci/actions/fleet-status@84d68a04c25cbc7c50598b05476293c4ae00a898 # main 2026-11-02, askalf/ci#159
with:
pr: ${{ github.event.pull_request.number || github.event.issue.number || github.event.workflow_run.pull_requests[0].number }}
head-repo: ${{ github.event.workflow_run.head_repository.full_name }}
Expand Down
Loading