flake: update to new digga api #300
Conversation
|
Would you agree we should wait the flatten tree / dns implementation and probably also |
Agreed. Don't think this should be merged until digga 0.2.0 is pretty much ready. |
| }; | ||
| }; | ||
|
|
||
| devshell.externalModules = { pkgs, ... }: { | ||
| packages = [ pkgs.agenix ]; |
There was a problem hiding this comment.
If we want to have the agenix command available in the devshell menu, we'd have to expose it as a commands = [ { } ];.
Not sure if that was the intention though, but it might be a good idea, regardless.
There was a problem hiding this comment.
Mainly wanted to avoid too much devshell related code in flake.nix. At some point I would like to add a devshell.toml in this repo, I just don't have anything to put in there other than agenix.
I would say to add the proper agenix command whenever a devshell.toml is added.
There was a problem hiding this comment.
At some point I would like to add a devshell.toml in this repo
👍
Pacman99
force-pushed
the
update-digga
branch
2 times, most recently
from
269d8a5 to
8d34f09
Compare
|
bors try |
tryBuild succeeded: |
|
bors try |
tryBuild succeeded: |
No description provided.