Skip to content

Skip signed/jsweep/* branches in PR Description Updater#42296

Merged
pelikhan merged 2 commits into
mainfrom
copilot/deep-report-quick-win
Jun 29, 2026
Merged

Skip signed/jsweep/* branches in PR Description Updater#42296
pelikhan merged 2 commits into
mainfrom
copilot/deep-report-quick-win

Conversation

Copilot AI commented Jun 29, 2026

Copy link
Copy Markdown
Contributor

The PR Description Updater was deterministically failing on merged PRs from signed/jsweep/* branches, consuming agent time and tokens before exiting. This change short-circuits those runs at the workflow gate so the updater only runs for supported merged PR branches.

  • What changed

    • Added a branch exclusion to the workflow-level if: condition in pr-description-caveman.md.
    • Recompiled pr-description-caveman.lock.yml so the generated workflow carries the same guard in the effective activation path.
  • Why this matters

    • Prevents known-noisy auto-generated/signed jsweep branches from entering the agent flow.
    • Avoids wasted spend on runs that are not expected to produce a useful PR description update.
  • Guard added

    if: github.event.pull_request.merged == true && !startsWith(github.event.pull_request.head.ref, 'signed/jsweep/')
  • Scope

    • No prompt-body changes.
    • No behavior change for non-signed/jsweep/* merged PRs.

Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Copilot AI changed the title [WIP] Add branch exclusion for signed/jsweep in PR Description Updater Skip signed/jsweep/* branches in PR Description Updater Jun 29, 2026
Copilot AI requested a review from pelikhan June 29, 2026 17:11
@pelikhan pelikhan marked this pull request as ready for review June 29, 2026 17:31
Copilot AI review requested due to automatic review settings June 29, 2026 17:31
@pelikhan pelikhan merged commit a1a53af into main Jun 29, 2026
1 check passed
@pelikhan pelikhan deleted the copilot/deep-report-quick-win branch June 29, 2026 17:31

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

This PR prevents the PR Description Updater workflow from running on merged PRs whose head branch starts with signed/jsweep/, avoiding deterministic failures and wasted agent/runtime spend by gating activation earlier.

Changes:

  • Added a frontmatter if: guard to skip merged PRs from signed/jsweep/* branches.
  • Recompiled the generated workflow so the same guard is applied in the effective pre_activation and activation job conditions.
Show a summary per file
File Description
.github/workflows/pr-description-caveman.md Adds a workflow gate to exclude merged PRs from signed/jsweep/* branches before the agent flow starts.
.github/workflows/pr-description-caveman.lock.yml Carries the same exclusion into the compiled workflow’s pre_activation/activation conditions.

Review details

Tip

Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

  • Files reviewed: 2/2 changed files
  • Comments generated: 0
  • Review effort level: Low

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.

[deep-report] [quick-win] Skip signed/jsweep/* branches in PR Description Updater

3 participants