Skip to content

ci: subscribe to edited PR events so workflows re-fire after Graphite restacks#429

Merged
vanceingalls merged 1 commit intomainfrom
vance/ci-edited-pr-trigger
Apr 23, 2026
Merged

ci: subscribe to edited PR events so workflows re-fire after Graphite restacks#429
vanceingalls merged 1 commit intomainfrom
vance/ci-edited-pr-trigger

Conversation

@vanceingalls
Copy link
Copy Markdown
Collaborator

What

Brief description of the change.

Why

Why is this change needed?

How

How was this implemented? Any notable design decisions?

Test plan

How was this tested?

  • Unit tests added/updated
  • Manual testing performed
  • Documentation updated (if applicable)

…te restacks

Workflows that gate `pull_request` on `branches: [main]` are silently
skipped for the head-ref push that `gt submit` makes during a stack
restack: Graphite momentarily flips the PR's base off of `main` before
resetting it back, and GitHub does not re-evaluate `pull_request`
triggers on `base_ref_changed` events with the default activity types.

The result: required checks (Build, Typecheck, Test: runtime contract,
Windows render, Windows tests, Semantic PR title) never run for that
head SHA and the PR sits in BLOCKED forever — even when the head
matches a green commit on `main`. PR #396 hit this exact pattern after
#426 added the `branches: [main]` filters.

Adding `edited` to the activity types causes the workflow to re-fire
when the base is set back to `main`. The existing `branches: [main]`
filter still rejects intermediate stack PRs, preserving #426's intent
of skipping CI on stack PRs that aren't yet targeting `main`.
Copy link
Copy Markdown
Collaborator Author

This stack of pull requests is managed by Graphite. Learn more about stacking.

@vanceingalls vanceingalls merged commit 113f9ea into main Apr 23, 2026
20 checks passed
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.

2 participants