Skip to content

Commit cc7ec77

Browse files
committed
fix: simplify PR trigger condition for e2e workflow
* Removed unnecessary check for github.actor == github.triggering_actor * Removed redundant permission check for admin (since write permission is sufficient) * Streamlined condition to focus on repository write permission only
1 parent be8b4ac commit cc7ec77

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

.github/workflows/e2e.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -31,7 +31,7 @@ jobs:
3131
if: >
3232
github.event_name == 'schedule' ||
3333
github.event_name == 'workflow_dispatch' ||
34-
(github.event_name == 'pull_request_target' && (github.actor == github.triggering_actor && (github.event.repository.permissions.write == true || github.event.repository.permissions.admin == true)))
34+
(github.event_name == 'pull_request_target' && github.event.repository.permissions.write == true)
3535
3636
concurrency:
3737
group: ${{ github.workflow }}-${{ matrix.provider }}-${{ github.event.pull_request.number || github.ref_name }}

0 commit comments

Comments
 (0)