search

nix-community / napalm

Support for building npm packages in Nix and lightweight npm registry [maintainer=?]
MIT License
104 stars 17 forks source link

Clean up flake.nix #44

Closed cyntheticfox closed 1 year ago

cyntheticfox commented 1 year ago

Clean up the flake.nix file with some functions, change the outputs to use the new "default" output types, clean up corresponding flake template, and add a flake-template check.

Updated the flake.lock file as well to update ghc and have all checks pass.

Signed-off-by: David Houston houstdav000@gmail.com

jtojnar commented 1 year ago

Thanks. Though it would be better to have one commit for each individual change – the current patch is very hard to review.

cyntheticfox commented 1 year ago

I'll split up the major commits, though some are dependent on one-another.

jtojnar commented 1 year ago

Thanks. It should be sufficient to create a single PR for all the commits if they are related. The only requirement is that the commits themselves are atomic so that their diffs can be reviewed individually.