Skip to content

ci: Claude review reports green (not skipped) on Dependabot PRs - #227

Merged
thomasluizon merged 1 commit into
mainfrom
chore/dependabot-review-green
Jun 23, 2026
Merged

ci: Claude review reports green (not skipped) on Dependabot PRs#227
thomasluizon merged 1 commit into
mainfrom
chore/dependabot-review-green

Conversation

@thomasluizon

Copy link
Copy Markdown
Owner

Why

On Dependabot PRs the review workflow should report a green success with no review performed — a job-level skip shows as "skipped", not "success".

Change

Move the if: github.actor != 'dependabot[bot]' guard from the job to the claude-code-action step. The job runs (checkout succeeds), the review step is skipped → the review check is a clean green success, and no Claude review runs on bot PRs.

Edits claude-review.yml, so the Claude action self-skips on this PR → admin merge.

Paired with the same change in orbit-ui-mobile.

🤖 Generated with Claude Code

… not the job)

The review workflow should report success on Dependabot PRs while doing no review — not a skipped job. Move the dependabot guard from the job to the claude-code-action step: the job runs (checkout succeeds), the review step is skipped, so the review check is a green success with no review performed.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@sonarqubecloud

Copy link
Copy Markdown

@thomasluizon
thomasluizon merged commit e29264f into main Jun 23, 2026
7 checks passed
@thomasluizon
thomasluizon deleted the chore/dependabot-review-green branch June 23, 2026 19:58
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