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
colorz: init at 1.0.3 #80188
colorz: init at 1.0.3 #80188
Conversation
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.
should just be 2 commits:
maintainers: add skykanin
colorz: init at 1.0.3
pkgs/tools/misc/colorz/default.nix
Outdated
|
||
meta = with lib; { | ||
description = "Color scheme generator"; | ||
homepage = https://github.com/metakirby5/colorz; |
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.
homepage = https://github.com/metakirby5/colorz; | |
homepage = "https://github.com/metakirby5/colorz"; |
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.
oh, ok. I didn't use strings for the homepage url since this isn't done in examples from the nixpkgs manual either https://nixos.org/nixpkgs/manual/#chap-meta
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.
oh, we need to change that, an rfc was merged recent
818b52a
to
4332dd5
Compare
should just be 2 commits:
https://nixos.org/nixpkgs/manual/#submitting-changes-making-patches |
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.
LGTM
application shows usage
https://github.com/NixOS/nixpkgs/pull/80188
1 package built:
colorz
Motivation for this change
This PR introduces colorz a python based tool for generation color schemes from images
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)