Skip to content

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

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

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

Conversation

@leighmcculloch
Copy link
Copy Markdown
Member

@leighmcculloch leighmcculloch commented Jun 3, 2026

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 by default.

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:16
@github-project-automation github-project-automation Bot moved this to Backlog (Not Ready) in DevX Jun 3, 2026
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

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 removes the existing GitHub Actions workflow that ran automated Claude-based PR reviews, aligning with the stated goal of reducing noisy/over-frequent review automation while the workflow is iterated on.

Changes:

  • Delete .github/workflows/claude-review.yml to stop triggering Claude Review on PR events.
  • Remove the associated concurrency/permissions/job configuration tied to that workflow.

@leighmcculloch leighmcculloch merged commit 8c3dede into main Jun 4, 2026
222 checks passed
@github-project-automation github-project-automation Bot moved this from Backlog (Not Ready) to Done in DevX Jun 4, 2026
@leighmcculloch leighmcculloch deleted the leighmcculloch-patch-1 branch June 4, 2026 05:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

3 participants