Felix Uhl
Felix Uhl
I think this is a good and simple change. One thing I would suggest is adding a functional test for this. Have a look at [this section of the manual](https://nixos.org/manual/nix/unstable/contributing/testing#functional-tests)...
> @cole-h Does this look right to you? (It looks right to me on a quick read, but I'm not terribly familiar with the profile logic and I'm leery of...
This is still broken.
This might be related to #709
I have the same issue. After running the commands @matthewbauer recommended, I get different errors: ``` $ sudo -i nix-channel --add https://nixos.org/channels/nixpkgs-unstable nixpkgs $ sudo -i nix-channel --update nixpkgs warning:...
@fdendorfer This is the case for a single-user install, yes. Is that how you installed nix? The multi-user installer shouldn't be run with sudo (I'm not even sure it's possible),...
@GNRSN The [determinate installer is maintained here](https://github.com/DeterminateSystems/nix-installer) and is not official (though most people recommend it over the official installer for good reason), so if this was an issue with...
Very cool! Yes, checking the formatting in PRs automatically would be very nice, and this is good ground-work for that. Could you rebase the PR on current master?
> I think ideally we would replace the makeshift formatter script with one generated by a framework: Sure, that's nicer. But this PR would be good enough in my book...
Alright I rebased and actually ran the formatter now. This already illustrates why we need PR check for this. I would be interested in adding that as well, but I'm...