Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Support pull_request_target events #16

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

aximov
Copy link

@aximov aximov commented May 4, 2023

What this PR does / Why we need it

In #10, one of this action's workflows was modified to be triggered by the pull_request_target event, and similar changes have been made in other actions-ecosystem repositories as well, but this action does not actually support this event. Since pull_request_target is considered safer than pull_request and also allows execution from forks, I believe it would be more appropriate to update the action's code rather than the workflows.

Which issue(s) this PR fixes

Fixes no issues

@aximov aximov requested a review from micnncim May 4, 2023 14:18
@aximov aximov self-assigned this May 4, 2023
@rustycl0ck
Copy link

Similar to #12, but seems like this repo is unmaintained now maybe???

ping @micnncim

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants