serenity
serenity copied to clipboard
nix: clean up files, use good practices
- nix: clean up files
- nix: update docs to reflect the new setup
Hello!
One or more of the commit messages in this PR do not match the SerenityOS code submission policy, please check the lint_commits CI job for more details on which commits were flagged and why.
Please do not close this PR and open another, instead modify your commit message(s) with git commit --amend and force push those changes to update this PR.
I also moved the new nix command to the top, as it should be preferred.
Should be good to go then, I've applied the changes requested.
hm I don't like how we're declaring this as "good practice" that are in fact (in practice) considered experimental in Nix. The previous description also stated the difference between the v2 and v3 cli, this does not.
hm I don't like how we're declaring this as "good practice" that are in fact (in practice) considered experimental in Nix. The previous description also stated the difference between the v2 and v3 cli, this does not.
The new description mentions that the v2 cli is old and will use the host nixpkgs, I think that's pretty descriptive enough?
Defining flakes as good practice is controversial, but it's the general way the community is going towards as it is more reproducible. Companies also prioritize flake support. (See DetSys, Hercules CI, Garnix and a few more which are all aimed towards flakes)