Skip to content

Commit 900b4ed

Browse files
committed
Remove Jacob and Ben from error handling message
1 parent e338250 commit 900b4ed

File tree

1 file changed

+6
-2
lines changed

1 file changed

+6
-2
lines changed

.github/workflows/require-jump-anza.yml

Lines changed: 6 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -89,12 +89,16 @@ jobs:
8989
const hasJump = jumpApprovers.some(u => approved.includes(u));
9090
const hasAnza = anzaApprovers.some(u => approved.includes(u));
9191
92+
// Filter out 'benhawkins18' and 'jacobcreech' from error messages
93+
const jumpApproversForMsg = jumpApprovers.filter(u => u !== 'benhawkins18' && u !== 'jacobcreech');
94+
const anzaApproversForMsg = anzaApprovers.filter(u => u !== 'benhawkins18' && u !== 'jacobcreech');
95+
9296
// helpful failure message
9397
const missingMsgs = [];
9498
if (!hasJump)
95-
missingMsgs.push(`Jump approval missing, need one of: ${jumpApprovers.join(', ')}`);
99+
missingMsgs.push(`Jump approval missing, need one of: ${jumpApproversForMsg.join(', ')}`);
96100
if (!hasAnza)
97-
missingMsgs.push(`Anza approval missing, need one of: ${anzaApprovers.join(', ')}`);
101+
missingMsgs.push(`Anza approval missing, need one of: ${anzaApproversForMsg.join(', ')}`);
98102
99103
// PR-check summary
100104
core.summary

0 commit comments

Comments
 (0)