From 595eaa3df4f6436fc0c940c1d762fe0ea1ba4b4b Mon Sep 17 00:00:00 2001 From: Tore Martin Hagen Date: Fri, 9 Aug 2024 10:59:08 +0200 Subject: [PATCH] Fixed problem with single never alone --- bin/never_alone_get_failing_pull_requests.sh | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/bin/never_alone_get_failing_pull_requests.sh b/bin/never_alone_get_failing_pull_requests.sh index fae3d79ad..593504ab4 100755 --- a/bin/never_alone_get_failing_pull_requests.sh +++ b/bin/never_alone_get_failing_pull_requests.sh @@ -67,8 +67,8 @@ function get_failing_pull_requests # Read each pull-request entry and check it while IFS= read -r pr_data; do # Check for missing reviews or if that list is empty - reviews=$(echo "${pr_data}" | jq '.[0].reviews') - github_review_decision=$(echo "${pr_data}" | jq '.[0].reviewDecision') + reviews=$(echo "${pr_data}" | jq '.reviews') + github_review_decision=$(echo "${pr_data}" | jq '.reviewDecision') local compliant="false" if [ "$reviews" = "null" ]; then pr_data=$(echo $pr_data | jq '. += {"failure": "no pull-request"}') @@ -81,11 +81,11 @@ function get_failing_pull_requests failed_reviews+=("$pr_data") else # Loop over reviews and check that at least one approver is not the same as committer - pr_author=$(echo "${pr_data}" | jq '.[0].author.login') - reviews_length=$(echo "${pr_data}" | jq '.[0].reviews | length') + pr_author=$(echo "${pr_data}" | jq '.author.login') + reviews_length=$(echo "${pr_data}" | jq '.reviews | length') for i in $(seq 0 $(( reviews_length - 1 ))) do - review=$(echo "${pr_data}" | jq ".[0].reviews[$i]") + review=$(echo "${pr_data}" | jq ".reviews[$i]") state=$(echo "$review" | jq ".state") review_author=$(echo "$review" | jq ".author.login") if [ "$state" = '"APPROVED"' -a "${review_author}" != "${pr_author}" ]; then