From d9834f2f1a1a6c1dbfe7da28b4426664904f4ae3 Mon Sep 17 00:00:00 2001
From: Javier Bullrich <javier@bullrich.dev>
Date: Wed, 21 Feb 2024 16:40:33 +0100
Subject: [PATCH] renamed step

---
 .github/workflows/review-trigger.yml | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

diff --git a/.github/workflows/review-trigger.yml b/.github/workflows/review-trigger.yml
index 3129ba37860a2..7b88260d760c6 100644
--- a/.github/workflows/review-trigger.yml
+++ b/.github/workflows/review-trigger.yml
@@ -21,7 +21,7 @@ jobs:
       - name: Skip merge queue
         if: ${{ contains(github.ref, 'gh-readonly-queue') }}
         run: exit 0
-      - name: Fail on non author pushing
+      - name: Fail when author pushes new code
         # Require new reviews when the author is pushing and he is not a member
         if: | 
           github.event.action == 'synchronize' && 
@@ -30,10 +30,12 @@ jobs:
         run: |
           echo "SENDER: $SENDER ; AUTHOR:$AUTHOR ; ACTION:$ACTION"
           echo "::error::Project needs to be reviewed again"
+
+          # We get the list of reviewers who approved the PR
           REVIEWERS=$(gh api repos/${{ github.repository }}/pulls/${{ github.event.number }}/reviews \
           --jq '{reviewers: [.[] | select(.state == "APPROVED") | .user.login]}')
           
-          echo "Requesting reviewers: $REVIEWERS to repos/${{ github.repository }}/pulls/${{ github.event.number }}/requested_reviewers $R"
+          # We request them to review again
           echo $REVIEWERS | gh api --method POST repos/${{ github.repository }}/pulls/${{ github.event.number }}/requested_reviewers --input -
           exit 1
         env: