From 8bebf77dbdc819edebbc329f1bcdca79cafe50cb Mon Sep 17 00:00:00 2001 From: Chuan-kai Lin Date: Wed, 7 Dec 2022 10:12:01 -0800 Subject: [PATCH 1/2] update-required-checks.sh: fix argument handling --- .github/workflows/script/update-required-checks.sh | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/.github/workflows/script/update-required-checks.sh b/.github/workflows/script/update-required-checks.sh index d83098b3d3..2326c55b51 100755 --- a/.github/workflows/script/update-required-checks.sh +++ b/.github/workflows/script/update-required-checks.sh @@ -10,7 +10,7 @@ fi if [ "$#" -eq 1 ]; then # If we were passed an argument, use that as the SHA - GITHUB_SHA="$0" + GITHUB_SHA="$1" elif [ "$#" -gt 1 ]; then echo "Usage: $0 [SHA]" echo "Update the required checks based on the SHA, or main." From 4a5ad5af185095095ba8a89c81625ec054e3e533 Mon Sep 17 00:00:00 2001 From: Chuan-kai Lin Date: Wed, 7 Dec 2022 14:27:00 -0800 Subject: [PATCH 2/2] update-required-checks.sh: ignore check-expected-release-files --- .github/workflows/script/update-required-checks.sh | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/.github/workflows/script/update-required-checks.sh b/.github/workflows/script/update-required-checks.sh index 2326c55b51..dfb1fc0f5a 100755 --- a/.github/workflows/script/update-required-checks.sh +++ b/.github/workflows/script/update-required-checks.sh @@ -23,7 +23,7 @@ fi echo "Getting checks for $GITHUB_SHA" # Ignore any checks with "https://", CodeQL, LGTM, and Update checks. -CHECKS="$(gh api repos/github/codeql-action/commits/"${GITHUB_SHA}"/check-runs --paginate | jq --slurp --compact-output --raw-output '[.[].check_runs | .[].name | select(contains("https://") or . == "CodeQL" or . == "LGTM.com" or contains("Update") or contains("update") or contains("test-setup-python-scripts") | not)] | unique | sort')" +CHECKS="$(gh api repos/github/codeql-action/commits/"${GITHUB_SHA}"/check-runs --paginate | jq --slurp --compact-output --raw-output '[.[].check_runs | .[].name | select(contains("https://") or . == "CodeQL" or . == "LGTM.com" or . == "check-expected-release-files" or contains("Update") or contains("update") or contains("test-setup-python-scripts") | not)] | unique | sort')" echo "$CHECKS" | jq