Added check to bulk pr scripts to bail if no parameter is given when running the scripts (#51)

* Added check to bulk pr scripts to bail if no parameter is given when running the script.

All of these scripts require the branch name the prs are based on to work and will not do any work otherwise.

* Added 'set -euo pipefail' to all shell scripts.

Author: soenkeliebau

bulk-pr/pr_approve

@@ -1,11 +1,13 @@
 #!/usr/bin/env bash
+set -euo pipefail
+PR_BRANCH_NAME=${1:?Must provide name of PR branch to operate on.}
 source repos
 for product in "${products[@]}"; do
-  STATE=$(gh pr view $1  -R stackabletech/${product}-operator --jq '.state' --json state)
+  STATE=$(gh pr view ${PR_BRANCH_NAME}  -R stackabletech/${product}-operator --jq '.state' --json state)
   if [[ $STATE -eq "OPEN" ]]; then
     echo "Approving ${product}"
-    gh pr review $1 --approve -R stackabletech/${product}-operator
-    gh pr merge $1 -R stackabletech/${product}-operator
+    gh pr review ${PR_BRANCH_NAME} --approve -R stackabletech/${product}-operator
+    gh pr merge ${PR_BRANCH_NAME} -R stackabletech/${product}-operator
   else
     echo "Skipping ${product}, PR already closed"
   fi

bulk-pr/pr_checks

@@ -1,6 +1,8 @@
 #!/usr/bin/env bash
+set -euo pipefail
+PR_BRANCH_NAME=${1:?Must provide name of PR branch to operate on.}
 source repos
 for product in "${products[@]}"; do
-    gh pr checks $1 -R stackabletech/${product}-operator 
+    gh pr checks ${PR_BRANCH_NAME} -R stackabletech/${product}-operator
 done
 

bulk-pr/pr_close

@@ -1,9 +1,11 @@
 #!/usr/bin/env bash
+set -euo pipefail
+PR_BRANCH_NAME=${1:?Must provide name of PR branch to operate on.}
 source repos
 for product in "${products[@]}"; do
-    STATE=$(gh pr view $1  -R stackabletech/${product}-operator --jq '.state' --json state)
+    STATE=$(gh pr view ${PR_BRANCH_NAME}  -R stackabletech/${product}-operator --jq '.state' --json state)
     if [[ $STATE -eq "OPEN" ]]; then
-        gh pr close $1 -R stackabletech/${product}-operator
+        gh pr close ${PR_BRANCH_NAME} -R stackabletech/${product}-operator
     else
         echo "Skipping ${product}, PR already closed"
     fi

bulk-pr/pr_diffs

@@ -1,6 +1,8 @@
 #!/usr/bin/env bash
+set -euo pipefail
+PR_BRANCH_NAME=${1:?Must provide name of PR branch to operate on.}
 source repos
 for product in "${products[@]}"; do
-    gh pr diff $1 -R stackabletech/${product}-operator
+    gh pr diff ${PR_BRANCH_NAME} -R stackabletech/${product}-operator
 done
 

bulk-pr/pr_list

@@ -1,4 +1,6 @@
 #!/usr/bin/env bash
+set -euo pipefail
+PR_BRANCH_NAME=${1:?Must provide name of PR branch to operate on.}
 source repos
 # Disable gh paging for results.
 PAGER=

bulk-pr/pr_status

@@ -1,8 +1,10 @@
 #!/usr/bin/env bash
+set -euo pipefail
+PR_BRANCH_NAME=${1:?Must provide name of PR branch to operate on.}
 source repos
 for product in "${products[@]}"; do
-    STATE=$(gh pr view $1  -R stackabletech/${product}-operator --jq '.state' --json state)
-    gh pr checks $1 -R stackabletech/${product}-operator &> /dev/null
+    STATE=$(gh pr view ${PR_BRANCH_NAME}  -R stackabletech/${product}-operator --jq '.state' --json state)
+    gh pr checks ${PR_BRANCH_NAME} -R stackabletech/${product}-operator &> /dev/null
     status=$?
     echo "${product}(${STATE}): ${status}"
 done