Skip signed/jsweep/* branches in PR Description Updater#42296
Merged
Conversation
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 Jun 29, 2026
signed/jsweep/* branches in PR Description Updater
Contributor
There was a problem hiding this comment.
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 fromsigned/jsweep/*branches. - Recompiled the generated workflow so the same guard is applied in the effective
pre_activationandactivationjob 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
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
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
if:condition inpr-description-caveman.md.pr-description-caveman.lock.ymlso the generated workflow carries the same guard in the effective activation path.Why this matters
Guard added
Scope
signed/jsweep/*merged PRs.