ci: remove Claude cloud-review workflow (perma-failing — no API key) - #155
Open
Nic-dorman wants to merge 1 commit into
Open
ci: remove Claude cloud-review workflow (perma-failing — no API key)#155Nic-dorman wants to merge 1 commit into
Nic-dorman wants to merge 1 commit into
Conversation
The claude-review job fails on every PR because the ANTHROPIC_API_KEY repo secret is empty; a check that can never pass is noise. Not a required status check (ruleset requires only linear-link + pr-template). claude.yml (@claude mention responder) deliberately kept. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Linear issue
V2-1064 — https://linear.app/autonominetwork/issue/V2-1064
Risk tier
Compatibility
Semver impact
CI-only — no crate code touched; no release needed for this change.
Test evidence
Workflow-file deletion only. Evidence is this PR's own check run: no
claude-reviewcheck appears, and the requiredlinear-link+pr-templatechecks still run. Verifiedclaude-reviewis not in the branch ruleset's required status checks, so nothing can be left waiting on a check that never reports.New dependency
none
ADR
n/a
Mitigation / rollback
git revertrestores the workflow unchanged; reinstating it only makes sense once theANTHROPIC_API_KEYrepo secret is actually populated.What
Delete
.github/workflows/claude-code-review.yml— the auto-reviewclaude-reviewjob that runs on every PR. It fails on every run because theANTHROPIC_API_KEYrepo secret is empty, so every PR in the repo carries a permanent red check (visible on saorsa-core #153/#154 since 2026-08-21). A check that can never pass trains reviewers to ignore red.claude.yml(the@claudemention responder) is deliberately kept: it fires only on explicit mention, so it adds no per-PR noise. Whether to keep or drop it is a separate call — noted on V2-1064.Same change lands in the sibling repo under the same ticket.
🤖 Generated with Claude Code