tools: do not mask errors on multiple commit retrieval

In commit-queue.sh, the assignment to `commits` will succeed and the script will continue if one of the two `git` commands fails, even with `-e` set. Split it into three separate assignments so that failures in the `git` commands will be clearly logged and cause the script to exit with a failure status code. PR-URL: #41340 Reviewed-By: Tierney Cyren <[email protected]> Reviewed-By: Tobias Nießen <[email protected]> Reviewed-By: Franziska Hinkelmann <[email protected]>
nodejs · Feb 1, 2022 · 9f2365c · 9f2365c
1 parent bc48ed2
commit 9f2365c
Showing 1 changed file with 3 additions and 1 deletion.
diff --git a/tools/actions/commit-queue.sh b/tools/actions/commit-queue.sh
@@ -72,7 +72,9 @@ for pr in "$@"; do
   fi
 
   if [ -z "$MULTIPLE_COMMIT_POLICY" ]; then
-    commits="$(git rev-parse $UPSTREAM/$DEFAULT_BRANCH)...$(git rev-parse HEAD)"
+    start_sha=$(git rev-parse $UPSTREAM/$DEFAULT_BRANCH)
+    end_sha=$(git rev-parse HEAD)
+    commits="${start_sha}...${end_sha}"
 
     if ! git push $UPSTREAM $DEFAULT_BRANCH >> output 2>&1; then
       commit_queue_failed "$pr"