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
NixOS Manual: document assertions and warnings #29206
Conversation
@roberth, thanks for your PR! By analyzing the history of the files in this pull request, we identified @thatdocslady, @copumpkin and @ericsagnes to be potential reviewers. |
"Importing <nixpkgs/nixos/modules/programs/virtualbox.nix> is " | ||
+ "deprecated, please use `virtualisation.virtualbox.host.enable = true' " | ||
+ "instead." ]; | ||
config.virtualisation.virtualbox.host.enable = true; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Even though this is example is very concise, having the same structure here as below for assertions would be more clearer I think.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks a lot! Docs are always very welcome! Looks good to me except the comment.
552a924
to
5bb059b
Compare
Ok, I have simplified the assertion example and I have adapted the warning example to match the assertion example more closely. In doing so, the warning example is not a real world example any more. It turns out that they are not used very often. Most of the warnings seem to be deprecations, moves and such, which are modeled differently. |
Thanks! |
* NixOS Manual: document assertions and warnings * NixOS manual: re-wrap assertions text (cherry picked from commit 1b1fc65)
Motivation for this change
assertions and warnings in NixOS modules were not documented yet.
Things done
build-use-sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)