-
Notifications
You must be signed in to change notification settings - Fork 3
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
Allow GitHub Actions workflow trigger for pull requests #127
Comments
yep this is an area we need to think about more, could you link to an example where you use halfpipe for PRs? I'm interested in whether halfpipe makes sense for these kind of simpler/additional workflows or maybe we should be looking more at reusable workflows for standardisation. |
There is for example these two examples: Yes, maybe these reusable workflows could be a good fit. The workflows differ in the watched/ignored paths, injected environment variables/secrets, and the one tasks that is executed (some use docker compose, some directly a docker image&script. The latter could be moved to docker compose I assume, to reduce differences). |
Looks like we would not be able to provide a private shared pool of reusable workflows only accessible from within the SN org. :-\ from Access to reusable workflows — Reusing workflows - GitHub Docs
On the other hand in Overview — Reusing workflows - GitHub Docs it says:
And in that case this might be a good fit, yes! I guess we have to try to find out. 🙂 |
luckily https://github.com/springernature/ee-github-actions reusable deploy to snpaas workflow I think it has good potential, we're just in a spiking phase though, there are some quirks and limitations to learn about |
We have some halfpipe manifests that are used to create GH Action workflows to execute tests against a pull request.
Currently, halfpipe only supports the
push
event to trigger a run. For our use case, we need to manually change the generated workflow to usepull_request
event.It would be great if this could be somehow supported in halfpipe directly, as it makes it much easier to the developers to use only one "configuration language" (halfpipe).
The text was updated successfully, but these errors were encountered: