Skip to content

kitty: use scopes as arguments, remove all-packages.nix override#484591

Merged
isabelroses merged 1 commit intoNixOS:masterfrom
mdaniels5757:kitty-override
Jan 28, 2026
Merged

kitty: use scopes as arguments, remove all-packages.nix override#484591
isabelroses merged 1 commit intoNixOS:masterfrom
mdaniels5757:kitty-override

Conversation

@mdaniels5757
Copy link
Member

@mdaniels5757 mdaniels5757 commented Jan 28, 2026

Split from #474456.

This is a step towards #454525/#483820, which will help enable checking for additional by-name directories (e.g. Python) in nixpkgs-vet.

For more information, please see #483859 (comment).

Things done

@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. labels Jan 28, 2026
Copy link
Member

@ethancedwards8 ethancedwards8 left a comment

Choose a reason for hiding this comment

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

nice! super excited about all of this progress!

Copy link
Contributor

@kashw2 kashw2 left a comment

Choose a reason for hiding this comment

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

LGTM, Thanks!

@nixpkgs-ci nixpkgs-ci bot added 12.approvals: 2 This PR was reviewed and approved by two persons. 12.approved-by: package-maintainer This PR was reviewed and approved by a maintainer listed in any of the changed packages. labels Jan 28, 2026
@isabelroses isabelroses added this pull request to the merge queue Jan 28, 2026
Merged via the queue into NixOS:master with commit 1c1a54f Jan 28, 2026
36 of 38 checks passed
@mdaniels5757 mdaniels5757 deleted the kitty-override branch January 28, 2026 21:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

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: 2 This PR was reviewed and approved by two persons. 12.approved-by: package-maintainer This PR was reviewed and approved by a maintainer listed in any of the changed packages.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants