CIをトリガーする条件に pull_request を追加

[ken3ypa]

概要

• 「fork運用でPRを作成した際、CIの実行がApproveされても実行されない」という現象に対するPRです

やったこと

• CI をトリガーするタイミングに on: pull_request を追加し、push に branches master 条件を追加
  • fork運用でPRを出した際にもCIがトリガーされる & 二重で CI がトリガーされないようにしています

関連

• actions/checkout@v3 を使う #122
• ci test #123

[k1LoW]

Pull Requestが作成されたタイミングで on.push も発火してしまうので、on.push は デフォルトブランチのみに制限するのはどうでしょう？

Suggested change

	- push
	- pull_request
	push:
	branches:
	- master
	pull_request:

[ken3ypa]

ありがとうございます〜！
おっしゃる通り、push と pull_request.open の2回でCIをフックしてしまいそうなので、bc8c427 にて修正しました 🙌

[ken3ypa]

📝

• on: pull_request を追加してPRを作成したところ、pull_request を検知してCIが流れることを確認した
• fork 運用だと push は検知しない
• CIを流すには権限を持ってる方のApproveが必要
  • ref: https://docs.github.com/en/actions/managing-workflow-runs/approving-workflow-runs-from-public-forks

入れた差分

❯ git diff
diff --git a/.github/workflows/test.yml b/.github/workflows/test.yml
index dc07756..a23486e 100644
--- a/.github/workflows/test.yml
+++ b/.github/workflows/test.yml
@@ -1,6 +1,7 @@
 name: test
-on: push
-
+on:
+  - push
+  - pull_request
 jobs:
   test-all:
     runs-on: ubuntu-latest

[ku00]

LGTMです！