ci/request-reviews: Don't fail when there's too many reviewers

It's better than getting failed CI and emails: NixOS#371528 (comment) And fix a shellcheck lint
bhipple · Jan 20, 2025 · 06ee611 · 06ee611
1 parent 1be2382
commit 06ee611
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/ci/request-reviews/request-reviewers.sh b/ci/request-reviews/request-reviewers.sh
@@ -69,8 +69,8 @@ for user in "${!users[@]}"; do
 done
 
 if [[ "${#users[@]}" -gt 10 ]]; then
-    log "Too many reviewers (${!users[@]}), skipping review requests"
-    exit 1
+    log "Too many reviewers (${!users[*]}), skipping review requests"
+    exit 0
 fi
 
 for user in "${!users[@]}"; do