Skip to content

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

Merged
infinisil merged 1 commit intoNixOS:masterfrom
willbush:workflows/revert-by-name-checks-concurrency-group
Apr 24, 2024
Merged

Revert "workflows: add concurrency group to check-by-name workflow"#306430
infinisil merged 1 commit intoNixOS:masterfrom
willbush:workflows/revert-by-name-checks-concurrency-group

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 willbush requested a review from infinisil as a code owner April 24, 2024 01:33
@github-actions github-actions bot added the 6.topic: policy discussion Discuss policies to work in and around Nixpkgs label Apr 24, 2024
@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.

@ofborg ofborg bot added 10.rebuild-darwin: 0 This PR does not cause any packages to rebuild on Darwin. 10.rebuild-linux: 0 This PR does not cause any packages to rebuild on Linux. labels Apr 24, 2024
@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
@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

Labels

6.topic: policy discussion Discuss policies to work in and around Nixpkgs 10.rebuild-darwin: 0 This PR does not cause any packages to rebuild on Darwin. 10.rebuild-linux: 0 This PR does not cause any packages to rebuild on Linux.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants