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
zoxide: 0.4.1 -> 0.4.3 #92250
zoxide: 0.4.1 -> 0.4.3 #92250
Conversation
ajeetdsouza/zoxide@v0.4.1...v0.4.3 Also removes unnecessary `rec` and `stdenv`.
Reviewed points
Possible improvementsComments |
}: | ||
|
||
rustPlatform.buildRustPackage rec { |
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.
just curious, why do you prefer let
over rec
?
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'm trying to stay away from rec
as much as possible because I think it's cleaner. I seem to remember that I had (or read) a conversation about rec
vs let
on IRC a while back (I think it was with @adisbladis. If not, sorry for the ping), but I can't find it in my logs (it was probably in April or prior to the end of May). I believe it boiled down to rec
-less being faster to evaluate?
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.
Builds and runs successfully on macos
Result of 1 package built:- zoxide |
ajeetdsouza/zoxide@v0.4.1...v0.4.3
Also removes unnecessary
rec
andstdenv
.Motivation for this change
New release.
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)