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
fluent-gtk-theme: 2023-06-20 -> 2024-04-28 #307603
Conversation
@@ -23,13 +23,13 @@ lib.checkListOfEnum "${pname}: tweaks" [ "solid" "float" "round" "blur" "noborde | |||
|
|||
stdenvNoCC.mkDerivation rec { | |||
inherit pname; | |||
version = "2023-06-20"; | |||
version = "2024-04-28"; | |||
|
|||
src = fetchFromGitHub { | |||
owner = "vinceliuice"; | |||
repo = pname; |
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.
repo = pname; | |
repo = "fluent-gtk-theme"; |
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.
Done.
@@ -23,13 +23,13 @@ lib.checkListOfEnum "${pname}: tweaks" [ "solid" "float" "round" "blur" "noborde | |||
|
|||
stdenvNoCC.mkDerivation 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.
Can you remove the recursion and move to finalAttrs
?
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.
Done.
8c4eaf1
to
ee43d86
Compare
@@ -74,4 +74,4 @@ stdenvNoCC.mkDerivation rec { | |||
platforms = platforms.unix; | |||
maintainers = [ maintainers.romildo ]; |
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.
Remove with lib;
and use the components explicitly when needed.
For this particular line:
maintainers = [ maintainers.romildo ]; | |
maintainers = with lib.maintainers; [ romildo ]; |
The rest can be prepended with lib.
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.
Done.
ee43d86
to
7a1aa5e
Compare
Description of changes
Update to version 2024-04-28
Closes #307512
Things done
nix.conf
? (See Nix manual)sandbox = relaxed
sandbox = true
nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD"
. Note: all changes have to be committed, also see nixpkgs-review usage./result/bin/
)Add a 👍 reaction to pull requests you find important.