The current / default way of importing the impermanence home-manager module feels kinda "hacky".
Using a flake output directly (instead of importing a specific .nix file using a path) is imo much more cleaner.
Additionally, in regards to future nixos caching / lazy eval efforts, this might make things easier, although I'm no expert at this.
Anyhow, I don't think there are any downsides to this change.
Yes, I know impermanence.nixosModules.home-manager.impermanence currently works too, but that also feels "hacky" and is kinda unreadable.
since this has been stale for almost a month, I added a description to hopefully speed up review ;)
@talyz
Would be nice if this got merged, I'm kinda autistic about my config :D
Btw: appreciate your (and everyone else's) work on this module. Thanks!
Add flake output for home-manager module
The current / default way of importing the impermanence home-manager module feels kinda "hacky". Using a flake output directly (instead of importing a specific .nix file using a path) is imo much more cleaner. Additionally, in regards to future nixos caching / lazy eval efforts, this might make things easier, although I'm no expert at this. Anyhow, I don't think there are any downsides to this change.
Yes, I know
impermanence.nixosModules.home-manager.impermanence
currently works too, but that also feels "hacky" and is kinda unreadable.since this has been stale for almost a month, I added a description to hopefully speed up review ;) @talyz Would be nice if this got merged, I'm kinda autistic about my config :D Btw: appreciate your (and everyone else's) work on this module. Thanks!