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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

workflows: add concurrency group to check-by-name workflow #306072

Merged

Conversation

willbush
Copy link
Member

Description of changes

Noticed yesterday that multiple runs can happen at the same time.
On event that triggers the workflow for the matching concurrency group, any previous runs of the workflow in the same group will be cancelled.

Things done


Add a 馃憤 reaction to pull requests you find important.

On event that triggers the workflow for the matching concurrency group, any
previous runs of the workflow in the same group will be cancelled.
Copy link
Member

@infinisil infinisil left a comment

Choose a reason for hiding this comment

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

Briefly tested it in tweag#92, seems to work!

(this is necessary because this PR itself uses the workflow file from the base branch, so this file itself is not even evaluated)

@infinisil infinisil merged commit b219715 into NixOS:master Apr 23, 2024
21 checks passed
@willbush willbush deleted the workflow/check-by-name/concurrency-group branch April 24, 2024 00:32
@infinisil infinisil added this to the RFC 140 milestone Apr 24, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants