Skip to content

Only skip PR creation when one is actually open#16

Merged
p-linnane merged 1 commit into
mainfrom
fix-regenerate-pr-check
Jul 18, 2026
Merged

Only skip PR creation when one is actually open#16
p-linnane merged 1 commit into
mainfrom
fix-regenerate-pr-check

Conversation

@andrew

@andrew andrew commented Jul 18, 2026

Copy link
Copy Markdown
Member

gh pr view $BRANCH matches merged and closed PRs too, so after #8 was merged the regenerate workflow always reported "PR already open" and never created a new one. Switch to gh pr list --state open and check for output so only a currently open PR suppresses creation.

Copilot AI review requested due to automatic review settings July 18, 2026 01:18
@andrew
andrew requested a review from p-linnane July 18, 2026 01:18

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

Updates the regenerate workflow’s “PR already open” detection so it only suppresses PR creation when there is an currently open PR for the regeneration branch, avoiding false positives from closed/merged PRs.

Changes:

  • Replace gh pr view $BRANCH (matches closed/merged PRs) with gh pr list --state open and an output check to detect only open PRs.

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

@p-linnane
p-linnane merged commit 4aff419 into main Jul 18, 2026
6 checks passed
@andrew
andrew deleted the fix-regenerate-pr-check branch July 18, 2026 03:57
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