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

change pull_request to pull_request_target #201

Merged
1 commit merged into from
Oct 24, 2022

Conversation

jarmak-nv
Copy link
Contributor

The current action uses on: pull_request but for it to work successfully, it needs to be on: pull_request_target

https://docs.github.com/en/actions/using-workflows/events-that-trigger-workflows#pull_request_target

@jarmak-nv jarmak-nv added non-breaking Non-breaking change feature request New feature or request labels Oct 20, 2022
@jarmak-nv jarmak-nv requested a review from a team as a code owner October 20, 2022 20:42
@jarmak-nv jarmak-nv added bug Something isn't working review: ready and removed feature request New feature or request labels Oct 20, 2022
Copy link
Contributor

@mdemoret-nv mdemoret-nv left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great, was going to suggest this change myself.

@mdemoret-nv
Copy link
Contributor

@gpucibot merge

@ghost ghost merged commit a889138 into nv-morpheus:branch-22.11 Oct 24, 2022
@jarmak-nv jarmak-nv deleted the fix-pr-project-action branch January 23, 2024 16:29
This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working non-breaking Non-breaking change
Projects
No open projects
Status: Done
Development

Successfully merging this pull request may close these issues.

2 participants