Skip to content

Commit 0b08ea8

Browse files
nanasessclaude
andcommitted
ci: Dependabot 判定を github.actor から PR 作成者へ修正
前回の `if: github.actor != 'dependabot[bot]'` は意図どおり動かなかった。 github.actor は「実行を引き起こしたユーザー」を指すため、別 PR をマージして Dependabot PR が自動 rebase された際、実行が人間(マージした本人)に帰属し スキップされなかった。PR 作成者を見る pull_request.user.login で判定すれば、 誰が再実行を誘発しても Dependabot PR を安定して識別できる。 Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
1 parent 8f7b03f commit 0b08ea8

1 file changed

Lines changed: 6 additions & 1 deletion

File tree

.github/workflows/claude-code-review.yml

Lines changed: 6 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -16,7 +16,12 @@ jobs:
1616
# the actions/checkout SHA), which makes claude-code-action's OIDC token exchange
1717
# fail with "Workflow validation failed" because the workflow no longer matches the
1818
# version on the default branch. Skipping avoids that expected, unavoidable failure.
19-
if: github.actor != 'dependabot[bot]'
19+
#
20+
# Use the PR author (pull_request.user.login), NOT github.actor: github.actor is
21+
# whoever triggered the run, so when merging another PR causes Dependabot to rebase
22+
# this one, the run is attributed to the human merger and the check would not be
23+
# skipped. The PR author stays 'dependabot[bot]' regardless of who triggers the run.
24+
if: github.event.pull_request.user.login != 'dependabot[bot]'
2025

2126
# Optional: Filter by PR author
2227
# if: |

0 commit comments

Comments
 (0)