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 event #50

Merged
merged 3 commits into from
Mar 20, 2024
Merged

Conversation

kuhnroyal
Copy link
Contributor

The pull_request_target is used for pull requests originating from forks with read only permissions by default.
This is required to use marocchino/sticky-pull-request-comment in PRs from forks. See marocchino/sticky-pull-request-comment#930

The pull_request_target is used for pull requests originating from forks with read only permissions by default.
@tm1000
Copy link
Member

tm1000 commented Mar 20, 2024

@kuhnroyal Thanks!!

@kuhnroyal
Copy link
Contributor Author

I will try to fix the checks later. This was just a quick hack in the web UI.

@tm1000 tm1000 merged commit b591154 into clearlyip:main Mar 20, 2024
1 check failed
@tm1000
Copy link
Member

tm1000 commented Mar 20, 2024

@kuhnroyal I fixed it so all good

Thanks again!

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

Successfully merging this pull request may close these issues.

None yet

2 participants