Skip to content

ci: stop re-running Claude review on same-commit PR events #77

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
emmax86 merged 1 commit into from
Jun 16, 2026
Merged

ci: stop re-running Claude review on same-commit PR events #77

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
6 changes: 5 additions & 1 deletion .github/workflows/claude-code-review.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,11 @@ name: Claude Code Review

on:
pull_request:
types: [opened, synchronize, ready_for_review, reopened]
# Only events that introduce a new/changed commit. Draft<->ready and
# reopen transitions fire on the same HEAD SHA as the prior run, so
# reviewing them again just re-reviews an identical diff. Drafts are
# still reviewed (on open and on each push).
types: [opened, synchronize]

@chatgpt-codex-connector chatgpt-codex-connector Bot Jun 16, 2026

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P2 Badge Keep reopened reviews for changed closed PRs

When a PR is closed, collaborators can still push new commits to its head branch and then reopen it; that reopen emits a pull_request reopened activity rather than an opened event, and there may have been no synchronize run while the PR was closed. With the workflow now limited to opened and synchronize, that reopened PR can either merge with no Claude review for the new HEAD or remain blocked by a missing required check, so reopened still needs to be included or handled with a same-SHA guard.

Useful? React with 👍 / 👎.

# Optional: Only run on specific file changes
# paths:
# - "src/**/*.ts"
Expand Down
Loading