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
tldr: 1.2.0 -> 1.3.0 #23465
tldr: 1.2.0 -> 1.3.0 #23465
Conversation
@c0bw3b, thanks for your PR! By analyzing the history of the files in this pull request, we identified @nckx and @Radvendii to be potential reviewers. |
pkgs/tools/misc/tldr/default.nix
Outdated
@@ -18,10 +18,10 @@ stdenv.mkDerivation rec { | |||
|
|||
meta = with stdenv.lib; { | |||
description = "Simplified and community-driven man pages"; | |||
longDescription = '' | |||
longDescription = " |
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.
Why this change?
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.
Note that it makes the indentation remain in the string...
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.
It was clearer to me that way because of the single quote in the description.
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.
I see. It is customary to use indented strings for long descriptions, however.
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.
Edited
Now we have two commits. Please squash them! |
Corrected longDescription
Squashed. |
pkgs/tools/misc/tldr/default.nix
Outdated
@@ -18,10 +18,10 @@ stdenv.mkDerivation rec { | |||
|
|||
meta = with stdenv.lib; { | |||
description = "Simplified and community-driven man pages"; | |||
longDescription = '' | |||
longDescription = " |
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.
Note that it makes the indentation remain in the string...
I squash-merged anyway... |
Motivation for this change
Package update
Things done
(nix.useSandbox on NixOS,
or option
build-use-sandbox
innix.conf
on non-NixOS)
nix-shell -p nox --run "nox-review wip"
./result/bin/
)