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: small formatting improvements #105969
Conversation
nixpkgs on patch-7 [!]
❯ nix-build nixos/release.nix -A manual.x86_64-linux <<<
these derivations will be built:
/nix/store/xqmq3ajrdqycsnp8p5fzk39aw4lkwmjv-generated-docbook.drv
/nix/store/sjy1kb3lhfhij4hnyc6n24r5apdjlssz-nixos-manual-combined.drv
/nix/store/595ynw127rfaz4fvvijqh7rzlh10g43i-manual-olinkdb.drv
/nix/store/6r64y841xj1wgy0hpp29vf1hls96dg4r-nixos-manual-html.drv
building '/nix/store/xqmq3ajrdqycsnp8p5fzk39aw4lkwmjv-generated-docbook.drv'...
building '/nix/store/sjy1kb3lhfhij4hnyc6n24r5apdjlssz-nixos-manual-combined.drv'...
manual-combined.xml validates
man-pages-combined.xml validates
building '/nix/store/595ynw127rfaz4fvvijqh7rzlh10g43i-manual-olinkdb.drv'...
Writing /nix/store/l49yx34lzqxvjw6fd69vrbsi880zg056-manual-olinkdb/manual.db for book(book-nixos-manual)
building '/nix/store/6r64y841xj1wgy0hpp29vf1hls96dg4r-nixos-manual-html.drv'...
Writing options.html for appendix(ch-options)
Writing release-notes.html for appendix(ch-release-notes)
Writing index.html for book(book-nixos-manual)
/nix/store/vil7fsfxizmfnkmbn9ximi7svk18zr9j-nixos-manual-html
nixpkgs on patch-7 took 36s
❯ |
141cfa8
to
87d8fc0
Compare
@@ -93,23 +94,23 @@ services.httpd.extraModules = [ | |||
</para> | |||
<para> | |||
<screen> | |||
<prompt>$ </prompt> svn create REPO_NAME |
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.
The spaces are supposed to be in prompt so that it is not copied.
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.
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.
In the picture, you can see that the space before the command is selected. But the space is not a part of the command so it should not be copied.
To achieve that, the space needs to be inside the prompt:
<prompt>$ </prompt>fooo
instead of outside it like you have here:
<prompt>$</prompt> fooo
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.
got it, corrected.
87d8fc0
to
4c812ea
Compare
I marked this as stale due to inactivity. → More info |
Motivation for this change
mostly indentation fixes in ``` code blocks.
Things done
sandbox
innix.conf
on non-NixOS linux)nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
./result/bin/
)nix path-info -S
before and after)