Skip to content

fix(ci): fix auto-approve to check claude[bot] comment#583

Merged
JasonXuDeveloper merged 1 commit into
masterfrom
fix/auto-approve-and-scorecard
Jan 26, 2026
Merged

fix(ci): fix auto-approve to check claude[bot] comment#583
JasonXuDeveloper merged 1 commit into
masterfrom
fix/auto-approve-and-scorecard

Conversation

@JasonXuDeveloper
Copy link
Copy Markdown
Owner

Summary

  • Fix auto-approve workflow to check for claude[bot] PR comment containing "No issues found. Checked for bugs and CLAUDE.md compliance."
  • Add branch-protection annotation to scorecard config explaining why last_push_approval is disabled

Changes

  1. auto-approve.yml: Check PR comments from claude[bot] instead of check run output fields (which are not set by Claude Code Action)
  2. .scorecard.yml: Add annotation explaining branch-protection settings

Test plan

  • Auto-approve triggers when Claude review posts "No issues found" comment
  • PR gets approved automatically when both Claude review and Unity Tests pass

🤖 Generated with Claude Code

- Check PR comments from claude[bot] for 'No issues found' message
- Add branch-protection annotation to scorecard config explaining
  why last_push_approval is disabled (required for auto-approve bot)

Signed-off-by: JasonXuDeveloper - 傑 <jason@xgamedev.net>
@github-actions github-actions Bot added the ci label Jan 26, 2026
@claude
Copy link
Copy Markdown

claude Bot commented Jan 26, 2026

Code review

No issues found. Checked for bugs and CLAUDE.md compliance.

@github-actions
Copy link
Copy Markdown

Unity Test Results

EditMode: All tests passed
PlayMode: All tests passed

Unity Version: 2022.3.55f1
Project Path: UnityProject

✅ All tests passed! The PR is ready for review.

View workflow run

Click here to view the full workflow run

@JasonXuDeveloper JasonXuDeveloper merged commit 91d9ded into master Jan 26, 2026
18 checks passed
@JasonXuDeveloper JasonXuDeveloper deleted the fix/auto-approve-and-scorecard branch January 26, 2026 09:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant