Skip to content

ci/eval/compare/maintainers: disable aliases#454527

Merged
wolfgangwalther merged 2 commits intoNixOS:masterfrom
wolfgangwalther:ci-reintroduce-alias
Oct 22, 2025
Merged

ci/eval/compare/maintainers: disable aliases#454527
wolfgangwalther merged 2 commits intoNixOS:masterfrom
wolfgangwalther:ci-reintroduce-alias

Conversation

@wolfgangwalther
Copy link
Contributor

We should never try to ping maintainers through package aliases, this can only lead to errors. One example case is, where an attribute is a throw alias, but then re-introduced in a PR. This would trigger the throw. By disabling aliases, we can fallback gracefully.

Should resolve #453264 (comment).

Things done


Add a 👍 reaction to pull requests you find important.

We should never try to ping maintainers through package aliases, this
can only lead to errors. One example case is, where an attribute is a
throw alias, but then re-introduced in a PR. This would trigger the
throw. By disabling aliases, we can fallback gracefully.
We have seen API errors in the prepare scripts frequently the last
couple of days. A quick fix should be to retry these requests.
@nixpkgs-ci nixpkgs-ci 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-25.05 labels Oct 22, 2025
@wolfgangwalther
Copy link
Contributor Author

Merging, because I see more and more PRs failing in prepare, which is hopefully fixed by the second commit here.

@wolfgangwalther wolfgangwalther added this pull request to the merge queue Oct 22, 2025
Merged via the queue into NixOS:master with commit 64017f7 Oct 22, 2025
62 of 66 checks passed
@wolfgangwalther wolfgangwalther deleted the ci-reintroduce-alias branch October 22, 2025 12:19
@nixpkgs-ci
Copy link
Contributor

nixpkgs-ci bot commented Oct 22, 2025

Successfully created backport PR for release-25.05:

@github-actions github-actions bot added the 8.has: port to stable This PR already has a backport to the stable release. label Oct 22, 2025
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.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant