Skip to content

Delete .github/workflows/claude-review.yml#1887

Merged
leighmcculloch merged 1 commit into
mainfrom
leighmcculloch-patch-1
Jun 3, 2026
Merged

Delete .github/workflows/claude-review.yml#1887
leighmcculloch merged 1 commit into
mainfrom
leighmcculloch-patch-1

Conversation

@leighmcculloch

@leighmcculloch leighmcculloch commented Jun 3, 2026

Copy link
Copy Markdown
Member

What

Delete .github/workflows/claude-review.yml.

Why

It needs some iteration. Fires too frequently. It's noisy and too verbose. There's no way to silence it on long iterative PRs. And we should iterate on the pull request target usage as well.

In the interim we've enabled Codex review, which does not have many of these downsides.

The shared claude-review workflow we can keep iterating on, and test it on one repo before bringing it to a bunch and creating too much noise:

Copilot AI review requested due to automatic review settings June 3, 2026 13:15
@leighmcculloch leighmcculloch requested a review from mootz12 June 3, 2026 13:15

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 pull request removes the GitHub Actions workflow that automatically ran the “Claude Review” job on pull request events, reducing automated review noise while the workflow is iterated on.

Changes:

  • Deleted .github/workflows/claude-review.yml (the Claude-based PR review workflow).
  • Removes the associated concurrency, permissions, and anthropics/claude-code-action step configuration previously used for automated inline PR feedback.

@mootz12 mootz12 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.

For the next attempt, it could be nice if this fires when requested. We could use the comment trigger:

/claude-review {append additional prompt}

@leighmcculloch leighmcculloch added this pull request to the merge queue Jun 3, 2026
Merged via the queue into main with commit 0bf40d1 Jun 3, 2026
209 checks passed
@leighmcculloch leighmcculloch deleted the leighmcculloch-patch-1 branch June 3, 2026 14:33
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