-
Notifications
You must be signed in to change notification settings - Fork 174
Flakes support #103
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Flakes support #103
Conversation
0da0360
to
2eb2917
Compare
.github/workflows/ci.yaml
Outdated
with: | ||
name: pre-commit-hooks | ||
signingKey: '${{ secrets.CACHIX_SIGNING_KEY }}' | ||
- run: nix flake check |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I can also recommend nix flake show
to validate the flake output schema: https://github.com/Mic92/sops-nix/blob/4f384662a85804fa2bc1bc1f99e70bb468e76f88/.github/workflows/test-flakes.yml#L29
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks!
@domenkozar is there a documentation update needed here? I have the following snippet:
Using
I've been waiting for flakes support so I've been eager to update, just updated and now I get the error:
I've checked what the
I'm curious if I'm doing something wrong now? or |
You want to use |
I'm not seeing
when I try this, shall I create an issue rather than continuing on this PR? I think we should update the README to mention whatever solution we find as it still mentions |
I'll provide an example in README :) |
@domenkozar got it working, thanks! I realize my error was that I wasn't importing it as a flake... 🤦 sorry! |
Blocked on #102Blocked on NixOS/nixpkgs#120752