-
-
Notifications
You must be signed in to change notification settings - Fork 13.1k
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
config.allowAliases: Define as option #166792
config.allowAliases: Define as option #166792
Conversation
This simplifies usages and makes the default value consistent. In a few cases, the default value was interpreted to be `false`, but this is useless, because virtually nobody will explicitly set `allowAliases = true;`.
I think we should do this for all config options used in nixpkgs. And then generate documentation out of it like we do with the nixos modules. See also #56227 which takes it even further (though maybe a bit too far with use flags). |
https://hydra.nixos.org/eval/1753299#tabs-removed Seems this broke |
NixOS#166792 made the aliases work, which was unexpected for terraform-full. Thanks to zowoq for reporting NixOS#166792 (comment)
#166792 made the aliases work, which was unexpected for terraform-full. Thanks to zowoq for reporting #166792 (comment)
… config keys This was added in NixOS/nixpkgs#166792. Fixes #74.
This pull request has been mentioned on NixOS Discourse. There might be relevant details there: https://discourse.nixos.org/t/where-are-options-like-config-cudasupport-documented/17805/4 |
|
Motivation and description
This simplifies usages and makes the default value consistent.
In a few cases, the default value was interpreted to be
false
,but this is useless, because virtually nobody will explicitly
set
allowAliases = true;
.Things done
sandbox = true
set innix.conf
? (See Nix manual)nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD"
. Note: all changes have to be committed, also see nixpkgs-review usage./result/bin/
)nixos/doc/manual/md-to-db.sh
to update generated release notes