Skip to content

Run autoharness CI jobs only in merge queue or on dispatch (#730) - #1

Closed
Tianshu-Huang wants to merge 2 commits into
mainfrom
ci-autoharness-merge-queue
Closed

Tianshu-Huang wants to merge 2 commits into
mainfrom
ci-autoharness-merge-queue

Conversation

@Tianshu-Huang

Copy link
Copy Markdown
Owner

Test PR on my fork: checks that the 5 gated jobs show as skipped on pull_request. Not for merging.

@Tianshu-Huang Tianshu-Huang reopened this Oct 23, 2026
feliperodri added a commit to Kieran-Hulsman/verify-rust-std that referenced this pull request Nov 2, 2026
## Problem

Five CI jobs (autoharness, metrics, list, analyzer, build JSON) run on
every PR push and take about five runner-hours. Every PR waits on them.
model-checking#730 asks to run them only before merging.

## Fix

Add `if: merge_group || workflow_dispatch` to those five jobs.

Autoharness is a required check, so it cannot be skipped as a whole job:
a skipped job loses the `(ubuntu-latest)` part of its name and the
required check never turns green. So for that job the `if:` is on each
step. The job still runs, does nothing, and passes in seconds.

## Testing

On my fork, a manual run passed all 16 jobs
([run](https://github.com/Tianshu-Huang/verify-rust-std/actions/runs/40005615582)),
and a PR skipped the five jobs with the two autoharness required checks
green ([PR](Tianshu-Huang#1)).
Did not test `merge_group`.

By submitting this pull request, I confirm that my contribution is made
under the terms of the Apache 2.0 and MIT licenses.

---------

Co-authored-by: Felipe R. Monteiro <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant