Skip to content

Commit 5292ae3

Browse files
authored
fix: use pull_request_target to allow permissions (#37)
Use `pull_request_target` in favor of `pull_request` because of this issue we had outlined in this PR: masterpointio/terraform-aws-ssm-agent#40 PRs created from forks were not able to be ran against the test action because of the GITHUB_TOKEN permissions that was needed. The `pull_request_target` provides that. We have to manually approve any of those PR's to run the test, so we are covered on the security side. <!-- This is an auto-generated comment: release notes by coderabbit.ai --> ## Summary by CodeRabbit - **Chores** - Updated the workflow trigger for automated tests to improve pull request handling. <!-- end of auto-generated comment: release notes by coderabbit.ai -->
1 parent d4bb431 commit 5292ae3

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

.github/workflows/test.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,7 @@ on:
44
push:
55
branches:
66
- main
7-
pull_request:
7+
pull_request_target:
88

99
permissions:
1010
actions: read

0 commit comments

Comments
 (0)