tools: simplify commit-queue.sh merge command

The mergeUrl() function is only used in one place. It's a one-liner and inlining it makes the script easier to understand, I think. It also means no future shellcheck complaints about needing to add error handling. PR-URL: #41314 Reviewed-By: Luigi Pinca <[email protected]> Reviewed-By: Antoine du Hamel <[email protected]> Reviewed-By: Gerhard Stöbich <[email protected]>
nodejs · Jan 14, 2022 · 9c41247 · 9c41247
1 parent 137c814
commit 9c41247
Showing 1 changed file with 1 addition and 5 deletions.
diff --git a/tools/actions/commit-queue.sh b/tools/actions/commit-queue.sh
@@ -12,10 +12,6 @@ DEFAULT_BRANCH=master
 COMMIT_QUEUE_LABEL="commit-queue"
 COMMIT_QUEUE_FAILED_LABEL="commit-queue-failed"
 
-mergeUrl() {
-  echo "repos/${OWNER}/${REPOSITORY}/pulls/${1}/merge"
-}
-
 commit_queue_failed() {
   pr=$1
 
@@ -91,7 +87,7 @@ for pr in "$@"; do
       --arg head "$(grep 'Fetched commits as' output | cut -d. -f3 | xargs git rev-parse)" \
       '{merge_method:"squash",commit_title:$title,commit_message:$body,sha:$head}' > output.json
     cat output.json
-    if ! gh api -X PUT "$(mergeUrl "$pr")" --input output.json > output; then
+    if ! gh api -X PUT "repos/${OWNER}/${REPOSITORY}/pulls/${pr}/merge" --input output.json > output; then
       commit_queue_failed "$pr"
       continue
     fi