Skip to content

Commit 45b8357

Browse files
committed
Add explicit handling for non-trusted permission levels
Address review feedback: explicitly handle the fallthrough case in the permission check to make it clear that triage, read, none, and API errors are not trusted signals. Made-with: Cursor
1 parent 081ff96 commit 45b8357

File tree

1 file changed

+3
-0
lines changed

1 file changed

+3
-0
lines changed

.github/workflows/restricted-paths-guard.yml

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -123,6 +123,9 @@ jobs:
123123
LABEL_ACTION="not needed (collaborator permission is a trusted signal)"
124124
TRUSTED_SIGNALS="collaborator_permission:$COLLABORATOR_PERMISSION"
125125
;;
126+
*)
127+
# triage, read, none, or API error: not a trusted signal
128+
;;
126129
esac
127130
fi
128131

0 commit comments

Comments
 (0)