🎯

GitHub Actionsで差分に応じたジョブの選択実行とPRマージにおける必須ジョブ設定を両立させる

2023/12/07に公開

GitHub

GitHub Actions

tech

本記事では、GitHub Actions 利用時における、PR の差分に基づくジョブの選択実行と、PR マージ時の必須ジョブ設定を両立させる方法について紹介します。

通常、PR の差分に基づいてジョブを選択実行する際は、paths や paths-ignore フィルターが利用されます。
一方でこの方法では、PR マージ時の必須ジョブ設定と競合を起こしてしまうことがあります。

以下では、この問題の回避策を紹介します。

やり方の概要

paths や paths-ignore フィルターの代わりに、Changed Files アクションを利用します。

Changed Files アクションにてファイル差分を確認し、チェックジョブの影響範囲内のファイルが含まれていた場合は、if 構文によりチェックジョブを実行します。

一方で、チェックジョブの影響範囲内のファイルが含まれていなかった場合は、同じく if 構文により何もしないというジョブを実行します。

さらに、PR マージ時の必須ジョブ設定では、上記のチェックジョブと何もしないジョブを両方とも必須として設定します。

これにより、不要なチェック実行を避けつつ、必要なステータスチェックを確実に行うことが可能になります。

背景となる課題の詳細

GitHub Actions では、paths や paths-ignore フィルターを使って、PR の差分に応じてワークフローを実行するかどうかを選択できます。

この機能により、影響を受けないワークフローやジョブをスキップし、GitHub Actions の実行時間とコストを削減できます。

また、「ステータスチェック必須」設定を用いて、特定のワークフローやジョブの完了を PR のマージ条件に設定できます。

この機能により、安全な開発プロセスを確保できます。

しかし、paths フィルターや paths-ignore フィルターを利用すると、スキップされたチェックジョブが「待機状態」となり、GitHub が「マージ不可」と判断する問題が生じます。

例えば、以下のように Test API というジョブを PR のマージ条件に設定します。

この状況下で、paths フィルターや paths-ignore フィルターにより Test API ジョブがスキップされると、必要なチェックステータスが待機状態のままとなりマージがブロックされます。

やり方の詳細

以下では具体的な回避策を紹介します。

前提となるチェックジョブ

たとえば、以下のような Python のテスト実行 CI が設定されているとします。

.github/workflows/ci_api.yaml

name: CI / API

on:
  pull_request:
    branches:
      - "main"
    paths:
      - "functions/**"

jobs:
  test:
    name: Test API
    runs-on: ubuntu-latest
    steps:
      - uses: actions/checkout@v4
      - name: Install Python dependencies
        run: pip install -r requirements.txt
      - name: Test
        run: pytest

paths フィルターを changed-files アクションに差し替える

ここで、paths フィルターを削除し、変更があるかどうかを判定する check-impact ジョブを追加します。

.github/workflows/ci_api.yaml

on:
  pull_request:
    branches:
      - "main"
-    paths:
-      - "functions/**"

jobs:
+  check-impact:
+    name: Check impact
+    runs-on: ubuntu-latest
+    outputs:
+      has-changed-related-files: ${{ steps.check-related-files.outputs.any_changed == 'true' }}
+    steps:
+      - uses: actions/checkout@v4
+        with:
+          fetch-depth: 0
+      - name: Check related files
+        id: check-related-files
+        uses: tj-actions/changed-files@v40
+        with:
+          files: "functions/**"
# ...

ジョブの発動条件を if に差し替え、何もしないジョブを追加する

if 構文を用いて、関連ファイルに変更がある場合のみテストジョブを実行し、そうでない場合は何もしない Test API (no need) ジョブを実行します。

.github/workflows/ci_api.yaml

  test:
    # ...
    name: Test API
+    needs: check-impact
+    if: needs.check-impact.outputs.has-changed-related-files == 'true'
    runs-on: ubuntu-latest
    steps:
      - uses: actions/checkout@v4
      - name: Install Python dependencies
        run: pip install -r requirements.txt
      - name: Test
        run: pytest
+  test-no-need:
+    name: Test API (no need)
+    needs: check-impact
+    if: needs.check-impact.outputs.has-changed-related-files == 'false'
+    runs-on: ubuntu-latest
+    steps:
+      - name: Skip
+        run: echo "No changes in files related to API, skipping."