Skip to content

ci: disable Claude Code Review workflow - #187

Merged
l1a merged 1 commit into
mainfrom
chore/disable-claude-code-review
Aug 11, 2026
Merged

ci: disable Claude Code Review workflow#187
l1a merged 1 commit into
mainfrom
chore/disable-claude-code-review

Conversation

@l1a

@l1a l1a commented Aug 11, 2026

Copy link
Copy Markdown
Owner

What

Turns off the Claude Code Review GitHub Actions workflow for pull requests:

  • the pull_request trigger (opened, synchronize, ready_for_review, reopened) is commented out and replaced with workflow_dispatch, so the workflow still exists and can be started by hand;
  • the claude-review job additionally carries if: false, so even a manual dispatch is a no-op until that guard is removed.

Two independent switches rather than one, so re-enabling is a deliberate act and neither half can silently turn the review back on.

Why

The automated review ran on every PR and every subsequent push, adding a required-looking check and API cost without changing review outcomes on this repo — the just pr gate plus the 15-job CI matrix already cover the ground it was reporting on. Disabling in place (rather than deleting the file) keeps the wiring intact for a future opt-in.

Also in this PR

Standard per-PR hygiene from AGENTS.md §4:

  • retch-cli 0.6.16 → 0.6.17 (patch), Cargo.lock refreshed via cargo check
  • docs/retch.1 regenerated with just man (version footer only — one line)
  • NOTES.md Current State header bumped and a release-log entry added

Release note for tagging: v0.6.16 was tagged at 622cf3b (#184), but #185 and #186 merged afterwards with Cargo.toml still reading 0.6.16. So v0.6.17 rolls up three changes — #185 (portable Justfile recipes), #186 (logo height / audio normalisation / line wrapping) and this chore — and it is a two-crate publish, since retch-sysinfo 0.1.52 is not on crates.io yet (the index has 0.1.51).

Skipped deliberately, each a legitimate no-op for a CI-only change: README.md (no user-visible flag, config key, or feature), the GitHub wiki (all 8 pages checked — none documents the workflow list), and docs/retch.md / tldr (no CLI flags changed).

Test plan

  • just pr gate green end to end: version-bump check, NOTES.md header check, just man regen matches the committed page, cargo check leaves Cargo.lock clean
  • just checkcargo fmt --check, cargo clippy --workspace -- -D warnings, and cargo clippy --features graphics -- -D warnings
  • cargo test --workspace — 59 cli + 15 integration + 115 sysinfo, all green
  • cargo audit — no new advisories (only the pre-existing allowed paste RUSTSEC-2024-0436 unmaintained warning)
  • claude-code-review.yml parses as YAML, exposes exactly one trigger (workflow_dispatch), and its claude-review job resolves if: false
  • No claude-review check appears on this PR — the point of the change; verified on this PR: gh pr checks 187 lists 17 checks (CodeQL x3, audit, aur, 7x build, graphics-feature, full-test/build-release/release/nixpkgs skipping) and zero claude-review entries
  • Manual dispatch remains available once merged (gh workflow run "Claude Code Review" is registered on the default branch) — cannot be verified pre-merge, since workflow_dispatch is only offered from the default branch's copy of the file

The claude-review job no longer runs on pull requests: the
`pull_request` trigger is replaced with `workflow_dispatch` and the
job carries `if: false`, so it is off by default but can still be
invoked manually if wanted.

Bumps retch-cli 0.6.16 -> 0.6.17 (patch), refreshes Cargo.lock,
regenerates docs/retch.1 for the new version footer, and updates the
NOTES.md Current State header and release log.

Assisted-By: Claude Opus 5
@l1a
l1a merged commit 1177d19 into main Aug 11, 2026
18 of 20 checks passed
@l1a
l1a deleted the chore/disable-claude-code-review branch August 11, 2026 15:19
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.

1 participant