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

CI/GHA: cancel outdated CI runs on new PR changes #9533

Closed
wants to merge 1 commit into from

Conversation

mback2k
Copy link
Member

@mback2k mback2k commented Sep 17, 2022

Avoid letting outdated CI runs continue if a PR receives new changes. Outside a PR we let them continue running
by tying the concurrency to the commit hash instead.

Also only let one CodeQL or Hacktoberfest job run at a time.

Other CI platforms we use have this build in, but GitHub unfortunately neither by default nor with a simple option.

This saves CI resources and therefore a little energy.

@mback2k mback2k added the CI Continuous Integration label Sep 17, 2022
@mback2k mback2k self-assigned this Sep 17, 2022
@mback2k mback2k changed the title GHA: cancel outdated CI runs on new PR changes CI/GHA: cancel outdated CI runs on new PR changes Sep 17, 2022
Avoid letting outdated CI runs continue if a PR receives
new changes. Outside a PR we let them continue running
by tying the concurrency to the commit hash instead.

Also only let one CodeQL or Hacktoberfest job run at a time.

Other CI platforms we use have this build in, but GitHub
unfortunately neither by default nor with a simple option.

This saves CI resources and therefore a little energy.
@mback2k mback2k marked this pull request as ready for review Sep 18, 2022
@mback2k mback2k requested a review from cmeister2 as a code owner Sep 18, 2022
@mback2k mback2k requested a review from bagder Sep 18, 2022
bagder
bagder approved these changes Sep 19, 2022
Copy link
Member

@bagder bagder left a comment

I can't comment on the yaml content, but I approve of what you want to do wit this.

Copy link
Contributor

@cmeister2 cmeister2 left a comment

I did read the docs and it seems like this will work, so happy to approve 😅

@mback2k mback2k closed this in 4e68933 Sep 19, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CI Continuous Integration
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants