From 342b640b4b91be3d0e311bb722744530777a8a44 Mon Sep 17 00:00:00 2001 From: Kefu Chai Date: Fri, 7 Feb 2025 09:59:04 +0800 Subject: [PATCH] .github: skip license check when no relevant files changed Skip the license header check step in `check-license-header.yaml` workflow when no files with configured extensions were changed in the pull request. Previously, the workflow would fail in this case since the --files argument requires at least one file path: ``` check-license.py: error: argument --files: expected at least one argument ``` Add `if` condition to only run the check when steps.changed-files.outputs.files is not empty. Signed-off-by: Kefu Chai --- .github/workflows/check-license-header.yaml | 1 + 1 file changed, 1 insertion(+) diff --git a/.github/workflows/check-license-header.yaml b/.github/workflows/check-license-header.yaml index 6db2147e0a..7a4cd84688 100644 --- a/.github/workflows/check-license-header.yaml +++ b/.github/workflows/check-license-header.yaml @@ -28,6 +28,7 @@ jobs: echo "files=$(git diff --name-only --diff-filter=A ${{ github.event.pull_request.base.sha }} ${{ github.sha }} | tr '\n' ' ')" >> $GITHUB_OUTPUT - name: Check license headers + if: steps.changed-files.outputs.files != '' run: | .github/scripts/check-license.py \ --files ${{ steps.changed-files.outputs.files }} \