You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
This repository has been archived by the owner on Feb 10, 2025. It is now read-only.
First of all I want to dearly thank you about the project. It's exactly what I'm looking for.
I haven't set it up yet and I'm just trying to figure out the exact mechanics it uses by reading the source.
However the mixed nix/bash scripts.nix makes it a bit harder than what it could be - hence this issue.
Would it be possible to split the scripts in their own .sh files each and include them in the scripts.nix? If so, I'd be happy to do it and submit a PR, it's more of a question if you're open to it (or have even better ideas!)
password manager integration is a sensitive subject and easily auditable code can aid a lot
The text was updated successfully, but these errors were encountered:
Sure, that sounds good. The project was originally based on agenix, which does it this way, and I threw this project together for a hackathon so it was pretty hacked together.
But having separate .sh files will also let us run shellcheck and shfmt on them which is a great bonus.
Happy to look at a PR.
Sign up for freeto subscribe to this conversation on GitHub.
Already have an account?
Sign in.
First of all I want to dearly thank you about the project. It's exactly what I'm looking for.
I haven't set it up yet and I'm just trying to figure out the exact mechanics it uses by reading the source.
However the mixed nix/bash
scripts.nix
makes it a bit harder than what it could be - hence this issue.Would it be possible to split the scripts in their own
.sh
files each and include them in thescripts.nix
? If so, I'd be happy to do it and submit a PR, it's more of a question if you're open to it (or have even better ideas!)password manager integration is a sensitive subject and easily auditable code can aid a lot
The text was updated successfully, but these errors were encountered: