Skip to content

make checks run for contributor forks too#6174

Merged
aryairani merged 5 commits intotrunkfrom
ci/transcripts-before-push
Feb 21, 2026
Merged

make checks run for contributor forks too#6174
aryairani merged 5 commits intotrunkfrom
ci/transcripts-before-push

Conversation

@aryairani
Copy link
Contributor

Overview

The required checks weren't being run when a PR originated in a contributor fork (e.g. #6155 cc @stew)

side fixes:

  • add concurrency groups to a couple of workflows so they're not churning on out-of-date work
  • fix check scripts to pass flags through

Implementation approach and notes

Switch verify-proofs from on push to pull_request, but set the ref to the push ref and not the default merge ref.

Interesting/controversial decisions

Loose ends

I thought about splitting ci.yaml into a separate file for PRs vs merge-queue checks.

Test coverage

Hard to test CI before its merged :(

@aryairani aryairani requested a review from a team as a code owner February 21, 2026 18:18
@aryairani aryairani changed the title Ci/transcripts before push make checks run for contributor forks too Feb 21, 2026
@aryairani aryairani enabled auto-merge February 21, 2026 18:48
@aryairani aryairani added this pull request to the merge queue Feb 21, 2026
Merged via the queue into trunk with commit 7910e45 Feb 21, 2026
5 checks passed
@aryairani aryairani deleted the ci/transcripts-before-push branch February 21, 2026 19:05
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