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 September 18, 2022 06:55
@mback2k mback2k requested a review from bagder September 18, 2022 06:55
Copy link
Member

@bagder bagder left a comment

Choose a reason for hiding this comment

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

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

Choose a reason for hiding this comment

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

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
Development

Successfully merging this pull request may close these issues.

None yet

3 participants