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
Copy Markdown
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
Copy Markdown
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
Copy Markdown
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
Copy Markdown
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
Copy Markdown
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
Copy Markdown
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