Skip to content

ci: skip Claude review on back-merge PRs - #210

Merged
finalerock44 merged 2 commits into
devfrom
ci/skip-claude-review-back-merge
Oct 2, 2026
Merged

finalerock44 merged 2 commits into
devfrom
ci/skip-claude-review-back-merge

Conversation

@finalerock44

Copy link
Copy Markdown
Contributor

The release workflow's back-merge PR is opened by the release-please GitHub App, and claude-code-action refuses non-human actors, so the required claude-review check fails and blocks the merge (#197, #209). The PR only carries production code already reviewed through dev, so skip it like the release-please release PRs.

What & why

Type of change

  • fix — bug fix
  • feat — new feature
  • perf — performance improvement
  • refactor — code change that's neither a fix nor a feature
  • docs — documentation only
  • chore / ci / build / test — tooling, no user-facing change
  • Breaking change (title has ! or PR notes a BREAKING CHANGE:)

Checklist

  • PR title follows the Conventional Commits format (see comment above)
  • pnpm lint passes
  • pnpm typecheck passes
  • pnpm build passes
  • I have not bumped the version or edited CHANGELOG.md (release-please handles this)
  • I have signed the CLA (the bot will prompt on first contribution)
  • Docs / README.md / STYLE_GUIDE.md updated if behaviour or output changed

How to test

The release workflow's back-merge PR is opened by the release-please
GitHub App, and claude-code-action refuses non-human actors, so the
required claude-review check fails and blocks the merge (#197, #209).
The PR only carries production code already reviewed through dev, so
skip it like the release-please release PRs.

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
@finalerock44 finalerock44 self-assigned this Oct 2, 2026
@finalerock44
finalerock44 merged commit 9f883a0 into dev Oct 2, 2026
9 checks passed
@finalerock44
finalerock44 deleted the ci/skip-claude-review-back-merge branch October 2, 2026 17:06
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