Skip to content

Fix Claude review for release bot PRs#25

Merged
hindermath merged 1 commit into
mainfrom
fix/allow-release-please-claude-review
Jun 18, 2026
Merged

Fix Claude review for release bot PRs#25
hindermath merged 1 commit into
mainfrom
fix/allow-release-please-claude-review

Conversation

@hindermath

Copy link
Copy Markdown
Owner

Summary

  • allow the github-actions bot slug for the automatic Claude Code Review workflow
  • document the small agent-driven workflow change in docs/project-statistics.md

Evidence

  • git diff --check
  • ruby -e "require 'yaml'; YAML.load_file('.github/workflows/claude-code-review.yml'); puts 'claude-code-review.yml OK'"

This addresses PR #23's failed Claude Code Review run: Workflow initiated by non-human actor: github-actions (type: Bot). Add bot to allowed_bots list or use '*' to allow all bots.

Copilot AI review requested due to automatic review settings June 18, 2026 11:11

Copilot AI left a comment

Copy link
Copy Markdown

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 unblocks the automated Claude Code Review GitHub Actions workflow for release PRs created by Release Please (which are authored by github-actions[bot]). It does so by explicitly allowing the github-actions bot slug to initiate the workflow, and it records that workflow/governance change in the project statistics ledger.

Changes:

  • Allow the github-actions bot via allowed_bots in the Claude Code Review workflow configuration.
  • Add bilingual (DE/EN) inline rationale comments for the bot exception in the workflow.
  • Update docs/project-statistics.md “Stand” date and append a corresponding ledger entry.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
.github/workflows/claude-code-review.yml Adds allowed_bots: 'github-actions' (with bilingual explanation) so Release Please PRs can trigger the Claude review job.
docs/project-statistics.md Updates the status line and appends a new chronologically latest ledger row documenting the workflow adjustment.

@hindermath hindermath enabled auto-merge (squash) June 18, 2026 11:19
@hindermath hindermath merged commit febde50 into main Jun 18, 2026
13 of 14 checks passed
@hindermath hindermath deleted the fix/allow-release-please-claude-review branch June 18, 2026 11:32
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