Skip to content

Commit 2571b08

Browse files
authored
Add check for PR status to approve, close and status scripts to better handle PRs that were already approved. (#27)
Removed regorule operator from list of repos to check for
1 parent 7bedfa1 commit 2571b08

File tree

4 files changed

+16
-6
lines changed

4 files changed

+16
-6
lines changed

bulk-pr/pr_approve

Lines changed: 8 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,12 @@
11
#!/usr/bin/env bash
22
source repos
33
for product in "${products[@]}"; do
4-
gh pr review $1 --approve -R stackabletech/${product}-operator
5-
gh pr review $1 --comment -b "bors r+" -R stackabletech/${product}-operator
4+
STATE=$(gh pr view $1 -R stackabletech/${product}-operator --jq '.state' --json state)
5+
if [[ $STATE -eq "OPEN" ]]; then
6+
echo "Approving ${product}"
7+
gh pr review $1 --approve -R stackabletech/${product}-operator
8+
gh pr review $1 --comment -b "bors r+" -R stackabletech/${product}-operator
9+
else
10+
echo "Skipping ${product}, PR already closed"
11+
fi
612
done

bulk-pr/pr_close

Lines changed: 6 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,10 @@
11
#!/usr/bin/env bash
22
source repos
33
for product in "${products[@]}"; do
4-
gh pr close $1 -R stackabletech/${product}-operator
4+
STATE=$(gh pr view $1 -R stackabletech/${product}-operator --jq '.state' --json state)
5+
if [[ $STATE -eq "OPEN" ]]; then
6+
gh pr close $1 -R stackabletech/${product}-operator
7+
else
8+
echo "Skipping ${product}, PR already closed"
9+
fi
510
done

bulk-pr/pr_status

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,8 +1,8 @@
11
#!/usr/bin/env bash
22
source repos
33
for product in "${products[@]}"; do
4+
STATE=$(gh pr view $1 -R stackabletech/${product}-operator --jq '.state' --json state)
45
gh pr checks $1 -R stackabletech/${product}-operator &> /dev/null
56
status=$?
6-
echo ${product}: ${status}
7+
echo "${product}(${STATE}): ${status}"
78
done
8-

bulk-pr/repos

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,6 @@
22
products=(
33
kafka
44
opa
5-
regorule
65
zookeeper
76
trino
87
druid

0 commit comments

Comments
 (0)