Skip to content

workflows/labels: count approving reviewers, not reviews#416668

Merged
wolfgangwalther merged 1 commit intoNixOS:masterfrom
wolfgangwalther:ci-fix-approvals
Jun 14, 2025
Merged

workflows/labels: count approving reviewers, not reviews#416668
wolfgangwalther merged 1 commit intoNixOS:masterfrom
wolfgangwalther:ci-fix-approvals

Conversation

@wolfgangwalther
Copy link
Contributor

@wolfgangwalther wolfgangwalther commented Jun 14, 2025

A single reviewer approving a Pull Request multiple times should only count once.

Example: #416076 - two approvals by make-42 resulted in an approvals: 2 label.

Things done


Add a 👍 reaction to pull requests you find important.

@github-actions github-actions 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. 6.topic: continuous integration Affects continuous integration (CI) in Nixpkgs, including Ofborg and GitHub Actions 6.topic: policy discussion Discuss policies to work in and around Nixpkgs backport release-24.11 labels Jun 14, 2025
@wolfgangwalther wolfgangwalther mentioned this pull request Jun 14, 2025
13 tasks
A single reviewer approving a Pull Request multiple times should only
count once.
Copy link
Member

@wamserma wamserma left a comment

Choose a reason for hiding this comment

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

diff LGTM

@github-actions github-actions bot added the 12.approvals: 1 This PR was reviewed and approved by one person. label Jun 14, 2025
@wolfgangwalther wolfgangwalther merged commit f917df3 into NixOS:master Jun 14, 2025
14 of 17 checks passed
@wolfgangwalther wolfgangwalther deleted the ci-fix-approvals branch June 14, 2025 14:34
@nixpkgs-ci
Copy link
Contributor

nixpkgs-ci bot commented Jun 14, 2025

Backport failed for release-24.11, because it was unable to cherry-pick the commit(s).

Please cherry-pick the changes locally and resolve any conflicts.

git fetch origin release-24.11
git worktree add -d .worktree/backport-416668-to-release-24.11 origin/release-24.11
cd .worktree/backport-416668-to-release-24.11
git switch --create backport-416668-to-release-24.11
git cherry-pick -x 2e033512487f35ea755c8a42116c420568b8e14c

@nixpkgs-ci
Copy link
Contributor

nixpkgs-ci bot commented Jun 14, 2025

Backport failed for release-25.05, because it was unable to cherry-pick the commit(s).

Please cherry-pick the changes locally and resolve any conflicts.

git fetch origin release-25.05
git worktree add -d .worktree/backport-416668-to-release-25.05 origin/release-25.05
cd .worktree/backport-416668-to-release-25.05
git switch --create backport-416668-to-release-25.05
git cherry-pick -x 2e033512487f35ea755c8a42116c420568b8e14c

@wolfgangwalther wolfgangwalther added the 8.has: port to stable This PR already has a backport to the stable release. label Jun 14, 2025
@wolfgangwalther
Copy link
Contributor Author

Backports added in #416409 and #416410

@wolfgangwalther
Copy link
Contributor Author

Unfortunately this failed for PRs with maintainers, so I quick-merged a fix in #416739.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

6.topic: continuous integration Affects continuous integration (CI) in Nixpkgs, including Ofborg and GitHub Actions 6.topic: policy discussion Discuss policies to work in and around Nixpkgs 8.has: port to stable This PR already has a backport to the stable release. 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. 12.approvals: 1 This PR was reviewed and approved by one person.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants