Skip to content

Comments

Direnv: Consider all .nix files (especially the ones in ./nix)#2864

Merged
smatting merged 1 commit intodevelopfrom
direnv-watch-nix-files-2
Nov 23, 2022
Merged

Direnv: Consider all .nix files (especially the ones in ./nix)#2864
smatting merged 1 commit intodevelopfrom
direnv-watch-nix-files-2

Conversation

@smatting
Copy link
Contributor

@smatting smatting commented Nov 23, 2022

This PR fixes a problem in direnv setup: .nix files in in the ./nix subdirectory were ignored for the computation of the cache-key: nix-rebuild. This is wrong because nix files in the ./nix directory define the environment.

@smatting smatting temporarily deployed to cachix November 23, 2022 17:55 Inactive
@smatting smatting temporarily deployed to cachix November 23, 2022 17:55 Inactive
@zebot zebot added the ok-to-test Approved for running tests in CI, overrides not-ok-to-test if both labels exist label Nov 23, 2022
@smatting smatting merged commit 6f479d4 into develop Nov 23, 2022
@smatting smatting deleted the direnv-watch-nix-files-2 branch November 23, 2022 17:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ok-to-test Approved for running tests in CI, overrides not-ok-to-test if both labels exist

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants