-
-
Notifications
You must be signed in to change notification settings - Fork 1.6k
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
Make install command in documentation compatible with fish shell #7474
Conversation
Shouldn't we just use
since that works in both shells? |
Skip the release notes for now, not sure we want to change these.
@edolstra That's much better, thanks! I've updated the PR and the description. I've also updated some other install commands I found, but left the release notes alone. Is it ok to update the release notes as well? |
@fricklerhandwerk @edolstra The nix.dev PR has been merged, but this one seems blocked on some tests that haven't run yet. Is there anything I can do to get the tests running? |
I was on vacation, sorry for the delay and thanks for hanging tight. Looks good, I'll merge this. |
No problem at all. Thanks for all your work on Nix! |
This pull request has been mentioned on NixOS Discourse. There might be relevant details there: https://discourse.nixos.org/t/nix-team-report-2022-10-2023-03/27486/1 |
This change has removed I'm new to Nix/NixOS, yet it seems like a lot of common internet advice expects Nix to be installed in multi-user mode. Even various Nix wiki pages mention editing That change also makes the paragraph following the install instructions make no sense:
There is no This change seems to have also missed the https://nixos.org/manual/nix/stable/installation/installation page, which still uses the |
See the description in NixOS/nix.dev/pull/422 for more context.