Skip to content

Fix CI not triggering on automated dependency update PRs#1086

Merged
stephentoub merged 2 commits intomainfrom
copilot/research-ci-workflow-triggers
Apr 16, 2026
Merged

Fix CI not triggering on automated dependency update PRs#1086
stephentoub merged 2 commits intomainfrom
copilot/research-ci-workflow-triggers

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Apr 16, 2026

PRs created by update-copilot-dependency.yml get no CI runs because GITHUB_TOKEN-triggered events are suppressed by GitHub to prevent recursive workflows.

Approach

Create the automated PR as a draft. A maintainer clicking "Ready for review" fires pull_request.ready_for_review from their own context, which is not suppressed. No secrets or GitHub Apps needed.

Changes

  • update-copilot-dependency.yml — added --draft to gh pr create
  • All 9 CI workflows — added types: [opened, synchronize, reopened, ready_for_review] to pull_request triggers (the first three are the existing defaults, made explicit)

Workflow

  1. Bot runs update-copilot-dependency, pushes branch, opens draft PR
  2. Maintainer reviews diff
  3. Maintainer clicks "Ready for review" → CI triggers
  4. Merge when green

Create dependency update PRs as drafts so maintainers can trigger
CI by clicking "Ready for review". Add ready_for_review to
pull_request types in all CI workflows.

Agent-Logs-Url: https://github.com/github/copilot-sdk/sessions/4e36e957-26c2-47a9-88c2-7b630ff3a33b

Co-authored-by: stephentoub <2642209+stephentoub@users.noreply.github.com>
@stephentoub stephentoub marked this pull request as ready for review April 16, 2026 02:36
@stephentoub stephentoub requested a review from a team as a code owner April 16, 2026 02:36
Copilot AI review requested due to automatic review settings April 16, 2026 02:36
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Ensures CI runs for PRs opened by the update-copilot-dependency.yml workflow, which currently creates PRs in a GITHUB_TOKEN context where downstream workflow triggers are suppressed by GitHub.

Changes:

  • Create automated dependency update PRs as drafts (gh pr create --draft) and add “next steps” guidance in the PR body.
  • Update CI workflows to explicitly trigger on pull_request event types including ready_for_review.
Show a summary per file
File Description
.github/workflows/update-copilot-dependency.yml Creates dependency update PRs as drafts and adds instructions to trigger CI via “Ready for review”.
.github/workflows/verify-compiled.yml Adds ready_for_review to PR trigger types so checks can run after draft PR is marked ready.
.github/workflows/scenario-builds.yml Same PR trigger type update to run scenario builds after “Ready for review”.
.github/workflows/python-sdk-tests.yml Same PR trigger type update for Python CI.
.github/workflows/nodejs-sdk-tests.yml Same PR trigger type update for Node.js CI.
.github/workflows/go-sdk-tests.yml Same PR trigger type update for Go CI.
.github/workflows/dotnet-sdk-tests.yml Same PR trigger type update for .NET CI.
.github/workflows/docs-validation.yml Same PR trigger type update for docs validation CI.
.github/workflows/corrections-tests.yml Same PR trigger type update for corrections tests CI.
.github/workflows/codegen-check.yml Same PR trigger type update for codegen verification CI.

Copilot's findings

  • Files reviewed: 10/10 changed files
  • Comments generated: 1

Comment thread .github/workflows/update-copilot-dependency.yml Outdated
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
@stephentoub stephentoub merged commit 97c10ae into main Apr 16, 2026
31 checks passed
@stephentoub stephentoub deleted the copilot/research-ci-workflow-triggers branch April 16, 2026 02:48
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.

3 participants