From 67bcd2ea8714f98e4fc0a0e8013aa17a4affbcb1 Mon Sep 17 00:00:00 2001 From: Mats Kindahl Date: Tue, 26 Nov 2024 10:39:45 +0100 Subject: [PATCH] Fix Hypercore PR approval Team agreed that changes to Hypercore table access method files only require a single approver for the time being. This might be revised in the future. --- .github/workflows/pr-approvals.yaml | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/.github/workflows/pr-approvals.yaml b/.github/workflows/pr-approvals.yaml index ca8edd561c0..349b29638b8 100644 --- a/.github/workflows/pr-approvals.yaml +++ b/.github/workflows/pr-approvals.yaml @@ -35,7 +35,10 @@ jobs: # Get the list of modified files in this pull request echo "Modified files: " gh pr view $PR_NUMBER --json files - files=$(gh pr view $PR_NUMBER --json files --jq '.files.[].path | select(startswith(".github") | not)') + # Get modified files, but exclude those that are workflow + # files or are related to Hypercore table access + # method. These require only a single reviewer. + files=$(gh pr view $PR_NUMBER --json files --jq '.files.[].path | select(startswith(".github") or test("hypercore|columnar_scan") | not)') # Get the number of approvals in this pull request echo "Reviews: "