Skip to content

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

Merged
leighmcculloch merged 1 commit into
masterfrom
remove-claude-review
Jun 3, 2026
Merged

Delete .github/workflows/claude-review.yml#5314
leighmcculloch merged 1 commit into
masterfrom
remove-claude-review

Conversation

@leighmcculloch

@leighmcculloch leighmcculloch commented Jun 3, 2026

Copy link
Copy Markdown
Member

Description

Remove the claude-review.yml.

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:

cc @anupsdf

Checklist

  • Reviewed the contributing document
  • Rebased on top of master (no merge commits)
  • Ran clang-format v8.0.0 (via make format or the Visual Studio extension)
  • Compiles
  • Ran all tests
  • If change impacts performance, include supporting evidence per the performance document

Copilot AI review requested due to automatic review settings June 3, 2026 13:10
@leighmcculloch leighmcculloch enabled auto-merge June 3, 2026 13:11

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 PR removes the claude-review.yml GitHub Actions workflow to disable the noisy Claude-based PR review automation while the workflow configuration and triggering behavior are iterated on.

Changes:

  • Deleted .github/workflows/claude-review.yml (the Claude Review pull_request_target workflow).
  • Removed the associated security-gating and checkout logic that enabled Claude reviews on certain fork PRs.

@leighmcculloch leighmcculloch added this pull request to the merge queue Jun 3, 2026
Merged via the queue into master with commit 51ecab1 Jun 3, 2026
73 of 74 checks passed
@leighmcculloch leighmcculloch deleted the remove-claude-review branch June 3, 2026 16:51
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