From 02a2c139a5c5f847decb346ef84be32677a69d65 Mon Sep 17 00:00:00 2001 From: Agent Date: Sun, 5 Apr 2026 13:49:34 +0000 Subject: [PATCH] fix: dispatcher.sh: || true suppresses errors in get_pr_merger / get_pr_reviews, making error handlers dead code (#189) --- docker/edge/dispatcher.sh | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/docker/edge/dispatcher.sh b/docker/edge/dispatcher.sh index 960123d..8b56343 100755 --- a/docker/edge/dispatcher.sh +++ b/docker/edge/dispatcher.sh @@ -159,7 +159,7 @@ get_pr_merger() { username: .merge_user?.login // .user?.login, merged: .merged, merged_at: .merged_at // empty - }' || true + }' } # Get PR reviews @@ -172,7 +172,7 @@ get_pr_reviews() { local ops_api="${FORGE_URL}/api/v1/repos/${FORGE_OPS_REPO}" curl -sf -H "Authorization: token ${FORGE_TOKEN}" \ - "${ops_api}/pulls/${pr_number}/reviews" 2>/dev/null || true + "${ops_api}/pulls/${pr_number}/reviews" 2>/dev/null } # Verify vault action was approved by an admin via PR review