Skip to content

Comments

lib.fileset.gitTracked,gitTrackedWith: Minor improvements#273664

Merged
infinisil merged 2 commits intoNixOS:masterfrom
tweag:fileset.gitTracked-improvements
Dec 13, 2023
Merged

lib.fileset.gitTracked,gitTrackedWith: Minor improvements#273664
infinisil merged 2 commits intoNixOS:masterfrom
tweag:fileset.gitTracked-improvements

Conversation

@infinisil
Copy link
Member

Description of changes

Just a refactoring and an improved error message for now.
This is motivated by a follow-up PR that should fix #269283 though :)

This work is sponsored by Antithesis

Things done

  • Added a test and ran tests successfully

Add a 👍 reaction to pull requests you find important.

Introduce an internal function for them to share more behavior.
This makes future changes easier.
@github-actions github-actions bot added the 6.topic: lib The Nixpkgs function library label Dec 11, 2023
@ofborg ofborg bot added 10.rebuild-darwin: 0 This PR does not cause any packages to rebuild on Darwin. 10.rebuild-linux: 1-10 This PR causes between 1 and 10 packages to rebuild on Linux. labels Dec 12, 2023
@infinisil infinisil merged commit c4e2e52 into NixOS:master Dec 13, 2023
@infinisil infinisil deleted the fileset.gitTracked-improvements branch December 13, 2023 04:14
@github-actions
Copy link
Contributor

Successfully created backport PR for release-23.11:

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

Labels

6.topic: lib The Nixpkgs function library 10.rebuild-darwin: 0 This PR does not cause any packages to rebuild on Darwin. 10.rebuild-linux: 1-10 This PR causes between 1 and 10 packages to rebuild on Linux.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

lib.fileset.gitTracked should support out-of-tree builds

1 participant