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

Reintroduce pull_request in GitHub workflow with if case #84

Merged
merged 1 commit into from
Dec 5, 2024

Conversation

agoscinski
Copy link
Collaborator

@agoscinski agoscinski commented Nov 28, 2024

I ran in PR #73 in the problem that the CI was not run, so I added this. I think aiida-core it does not show the pull_request events that are cancelled as in this PR, but I am not sure how it is done it there.

The disadvantage of removing the pull_request as trigger event for an GitHub workflow is that it does not trigger for pushes from a foreign repo. To run the workflow also in this case without running the workflow twice for PRs based on the original repo we introduce an if case that runs the pull_request event only when it from a foreign repo.

See https://wildwolf.name/github-actions-how-to-avoid-running-the-same-workflow-multiple-times/ for more explanation

The disadvantage of removing the `pull_request` as trigger event for an
GitHub workflow is that it does not trigger for pushes from a foreign
repo. To run the workflow also in this case without running the workflow
twice for PRs based on the original repo we introduce an if case that
runs the `pull_request` event only when it from a foreign repo.
@agoscinski agoscinski requested a review from eimrek November 28, 2024 07:32
Copy link
Member

@eimrek eimrek left a comment

Choose a reason for hiding this comment

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

look good!

@eimrek eimrek merged commit a2288e2 into master Dec 5, 2024
7 checks passed
@eimrek eimrek deleted the reintroduce-pr-event branch December 5, 2024 12:20
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.

2 participants