Skip to content

Commit

Permalink
Merge pull request #1103 from github/aeisenberg/fix-required-checks-s…
Browse files Browse the repository at this point in the history
…cript

Ensure there are no duplicates when sending up required checks
  • Loading branch information
aeisenberg committed Jun 17, 2022
2 parents 2e111b2 + 7adb33d commit 30fe0a5
Show file tree
Hide file tree
Showing 3 changed files with 4 additions and 2 deletions.
1 change: 1 addition & 0 deletions .github/workflows/expected-queries-runs.yml
Expand Up @@ -16,6 +16,7 @@ on:

jobs:
expected-queries:
name: Expected Queries Tests
timeout-minutes: 45
runs-on: ubuntu-latest
steps:
Expand Down
3 changes: 2 additions & 1 deletion .github/workflows/query-filters.yml
Expand Up @@ -15,7 +15,8 @@ on:
workflow_dispatch: {}

jobs:
expected-queries:
query-filters:
name: Query Filters Tests
timeout-minutes: 45
runs-on: ubuntu-latest
steps:
Expand Down
2 changes: 1 addition & 1 deletion .github/workflows/script/update-required-checks.sh
Expand Up @@ -21,7 +21,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") | not)] | 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 contains("Update") | not)] | unique | sort')"

echo "$CHECKS" | jq

Expand Down

0 comments on commit 30fe0a5

Please sign in to comment.