Skip to content

Remove claude.yml and claude-code-review.yml (will be re-added once I've corrected them)#1222

Merged
todthomson merged 1 commit into
mainfrom
eft/tod/fix/temp-remove-claude-github-actions
May 14, 2026
Merged

Remove claude.yml and claude-code-review.yml (will be re-added once I've corrected them)#1222
todthomson merged 1 commit into
mainfrom
eft/tod/fix/temp-remove-claude-github-actions

Conversation

@todthomson
Copy link
Copy Markdown
Contributor

@todthomson todthomson commented May 14, 2026

These were added in a recent PR: #1221

@borland made me realise that I had configured Claude to automatically do code reviews, when actually we only want to make it so that people can opt-in to these.

For safety, I'll remove them in this PR, and then use Claude Code CLI to re-add them (opt-in) in a 2nd PR.

@todthomson todthomson requested a review from a team as a code owner May 14, 2026 06:18
Copy link
Copy Markdown

@claude claude Bot left a comment

Choose a reason for hiding this comment

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

Claude Code Review

This repository is configured for manual code reviews. Comment @claude review to trigger a review and subscribe this PR to future pushes, or @claude review once for a one-time review.

Tip: disable this comment in your organization's Code Review settings.

@todthomson todthomson enabled auto-merge (squash) May 14, 2026 06:18
@todthomson todthomson changed the title temp: remove these files, so i can re-add them with the correct config Remove claude.yml and claude-code-review.yml (will be re-added once I've corrected them) May 14, 2026
@todthomson todthomson merged commit 4026cb9 into main May 14, 2026
46 of 50 checks passed
@todthomson todthomson deleted the eft/tod/fix/temp-remove-claude-github-actions branch May 14, 2026 07:13
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.

2 participants