-
Notifications
You must be signed in to change notification settings - Fork 21
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
Allow to specify some flake outputs from Nickel #140
Conversation
aa02bf2
to
283864e
Compare
Add project.ncl and nickel.lock.ncl, use them for this very flake. For now use only devshells. Update cachix/install-nix-action to install Nix 2.17 that doesn't evaluate `packages` for all systems on `nix flake check`. Preparation for dogfooding future improvements from #58
It's using old Nickel version that will be incompatible with future changes.
Add `flake` field to `project.ncl` that allows to specify certain flake outputs. For example, you can add lines like these to run `hello` as an app or a check: flake.apps.hello.program = nix-s%"%{import_nix "nixpkgs#hello"}/bin/hello"%, flake.checks.hello = import_nix "nixpkgs#hello", I will be gragually migrating apps and checks from our `flake.nix` to `project.ncl` using this. Part of #58
I'm not sure I get it. Why are some outputs at toplevel and some under |
@thufschmitt I’m not sure we want to expose everything at top level, so I’ve decided to namespace it. Mixing flake API with Organist one doesn’t feel right to me because it’s less declarative, less abstract and less typed. I feel like |
Fair enough. Let's keep our distances from the Nix interface for the time being. We are somewhat tied to it though since Nix is our entry point. Not sure how to best juggle this in the long run. |
Add
flake
field toproject.ncl
that allows to specify certain flake outputs. For example, you can add lines like these to runhello
as an app or a check:I will be gragually migrating apps and checks from our
flake.nix
toproject.ncl
using this.Also update flake inputs for the example to use newer Nickel required by the new changes.
Part of #58.