-
Notifications
You must be signed in to change notification settings - Fork 108
add github actions #480
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
add github actions #480
Conversation
7dcdb36
to
fd44bb3
Compare
570aecc
to
52fab07
Compare
ed62408
to
6cbdb9e
Compare
0068a77
to
5b7bb98
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I love that you made it a template.
~❓ Do you have a link to this running on a fork that I can see? ~
Oops, I see the link now. Looks good!
My main concern with this is that it seems like we are now duplicating a lot of the logic that lives in the repos and CI scripts in these workflows. However, I really want this so I'm not going to object 😄 Can we also add a job / step that checks if the generated workflow file is up-to-date? |
I am approving this PR, but I think this is a good idea for future improvement @maxmoehl. |
Summary
To perform basic validations before the PR is merged to develop branch.
Validated in the forked repo: kart2bc#1
https://github.com/kart2bc/routing-release/actions/runs/15077414452/job/42388907122
The gh action setting should be enabled in the cloudfoundry/routing-release repo.
Backward Compatibility
Breaking Change? No