Skip to content

fix: DependaBot以外のPRにチェックが表示される問題を修正#243

Merged
o-ga09 merged 3 commits into
mainfrom
fix/dependabot-auto-merge-trigger
May 16, 2026
Merged

fix: DependaBot以外のPRにチェックが表示される問題を修正#243
o-ga09 merged 3 commits into
mainfrom
fix/dependabot-auto-merge-trigger

Conversation

@o-ga09
Copy link
Copy Markdown
Owner

@o-ga09 o-ga09 commented May 16, 2026

close #

実装内容

問題

pull_request_target はすべてのPRでトリガーされるため、DependaBot以外のPRでも「Dependabot Auto Merge / auto-merge (skipped)」というチェックが表示されていた。

原因

GitHub Actionsの仕様上、ジョブの if 条件で false になりスキップされたジョブも、GitHub Checks APIに "neutral" ステータスとしてエントリが作られPRに表示される。

修正内容

pull_request_targetpush + dependabot/** ブランチパターンに変更した。

# 修正前
on:
  pull_request_target:
    types: [opened, synchronize, reopened]

# 修正後
on:
  push:
    branches:
      - 'dependabot/**'

これにより、DependaBotのブランチへのプッシュ時のみワークフローが実行され、他のPRには一切表示されない。

また、dependabot/fetch-metadata アクションは pull_request_target コンテキストが必要なため、代わりに git diff HEAD~1 -- go.mod から更新パッケージ名・バージョンを直接取得するよう変更した。

動作確認

エビデンス
  • DependaBotのPRでのみワークフローが実行されることを確認
  • go.mod の差分解析でパッケージ名とバージョンが正しく抽出されることを確認

テスト結果

  • DependaBotのPR作成時にワークフローが実行されることを確認
  • 通常のPRではワークフローが表示されないことを確認

pull_request_targetはすべてのPRでトリガーされるため、
DependaBot以外のPRにもskippedチェックが表示されていた。
pushイベント+dependabot/**ブランチパターンに変更し、
DependaBotのブランチへのpush時のみ実行されるようにした。
dependabot/fetch-metadataの代わりにgo.modの差分から
パッケージ情報を取得するよう変更。

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@github-actions

This comment has been minimized.

gh pr merge --autoはgit操作を内部で実行するため、
gitリポジトリコンテキストが解決できず
"fatal: not a git repository"エラーが発生していた。
--repo フラグでリポジトリを明示することで解消する。

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@github-actions

This comment has been minimized.

/migrateの絶対パス指定ではCMDと同じ./migrateで実行できない。
mainと同じパターンでカレントディレクトリにコピーすることで
./migrateで実行可能にする。

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@github-actions
Copy link
Copy Markdown
Contributor

Code Metrics Report

Coverage Code to Test Ratio Test Execution Time
26.9% 1:0.4 1m29s

Reported by octocov

@o-ga09 o-ga09 merged commit 82f790a into main May 16, 2026
7 checks passed
@o-ga09 o-ga09 deleted the fix/dependabot-auto-merge-trigger branch May 16, 2026 14:43
@github-actions github-actions Bot mentioned this pull request May 16, 2026
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