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

Revert "workflows: add concurrency group to check-by-name workflow" #306430

Conversation

willbush
Copy link
Member

@willbush willbush commented Apr 24, 2024

This reverts commit 75600cd.

Description of changes

@infinisil I don't think it's working as expected.


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

@willbush
Copy link
Member Author

Hmm maybe it does run after all queued checks complete. Still looking for an example of it being canceled with no other pending checks.

@infinisil
Copy link
Member

Huh weird, yeah let's revert for now, thanks for double-checking this!

@infinisil infinisil merged commit 6f94ae0 into NixOS:master Apr 24, 2024
22 of 23 checks passed
@infinisil
Copy link
Member

infinisil commented Apr 24, 2024

Ahh:

Canceling since a higher priority waiting request for 'check-by-name-refs/heads/master' exists

I guess we should use ${{ github.event.pull_request.number }} instead.

@willbush
Copy link
Member Author

Ahh:

Canceling since a higher priority waiting request for 'check-by-name-refs/heads/master' exists

I guess we should use ${{ github.event.pull_request.number }} instead.

Ah that makes sense!

@willbush willbush deleted the workflows/revert-by-name-checks-concurrency-group branch April 24, 2024 02:43
@infinisil
Copy link
Member

Btw I restarted all the cancelled builds manually (were only a couple), I think it should all be resolved now

infinisil added a commit to NixOS/nixfmt that referenced this pull request Apr 24, 2024
Turns out `github.ref` doesn't point to the pull request's ref when
using `pull_request_target`: NixOS/nixpkgs#306430
infinisil added a commit to NixOS/nixfmt that referenced this pull request Apr 25, 2024
Turns out `github.ref` doesn't point to the pull request's ref when
using `pull_request_target`: NixOS/nixpkgs#306430
infinisil added a commit to NixOS/nixfmt that referenced this pull request Apr 25, 2024
Turns out `github.ref` doesn't point to the pull request's ref when
using `pull_request_target`: NixOS/nixpkgs#306430
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