Skip to content

Conversation

@booxter
Copy link
Contributor

@booxter booxter commented May 1, 2025

pull_request_target means that the workflow will be pulled from main,
not from the PR under review; so it's a waste to run the workflow in
this case.

Signed-off-by: Ihar Hrachyshka [email protected]

pull_request_target means that the workflow will be pulled from main,
not from the PR under review; so it's a waste to run the workflow in
this case.

Signed-off-by: Ihar Hrachyshka <[email protected]>
@mergify mergify bot added the CI/CD Affects CI/CD configuration label May 1, 2025
Copy link
Contributor

@ktdreyer ktdreyer left a comment

Choose a reason for hiding this comment

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

The reasoning sounds right to me.

@booxter booxter requested a review from danmcp May 1, 2025 19:41
@mergify mergify bot added the one-approval label May 1, 2025
@mergify mergify bot merged commit fd03460 into main May 1, 2025
9 checks passed
@mergify mergify bot removed the one-approval label May 1, 2025
@mergify mergify bot deleted the ihrachyshka-dont-trigger-on-own-workflow-for-pull-request-targets branch May 1, 2025 20:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CI/CD Affects CI/CD configuration

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants