-
-
Notifications
You must be signed in to change notification settings - Fork 15.3k
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’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[Backport release-24.11] format and sort top-level/python-packages.nix #392723
[Backport release-24.11] format and sort top-level/python-packages.nix #392723
Conversation
Signed-off-by: Paul Meyer <katexochen0@gmail.com>
Let's backport the CI job as well. |
ec83674
to
6c12376
Compare
@wolfgangwalther Do we add the git-blame-ignores on release, too? |
I'm not sure what the convention here is. It's unlikely to cause conflicts, because those are always added in separate commits and never make sense to backport 1:1. So we could as well add them. But it would probably not give us much if that's not done for other commits like that, too. @infinisil WDYT? |
In any case, we can merge now to avoid merge conflicts from creeping in - we can always add the commit hashes later, if we want to. |
manual backport of #391087
Things done
nix.conf
? (See Nix manual)sandbox = relaxed
sandbox = true
nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD"
. Note: all changes have to be committed, also see nixpkgs-review usage./result/bin/
)Add a 👍 reaction to pull requests you find important.