-
-
Notifications
You must be signed in to change notification settings - Fork 14.6k
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
hellwal: init at 1.0.2 #364360
hellwal: init at 1.0.2 #364360
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.
Welcome to Nixpkgs! There's a few issues that need to be resolved
Also, commit & PR should be titled hellwal: init at 1.0.2
How about now? Is it fine? Oh. it's not... |
Thanks you all for helping me :D I think it's ready right? |
Needs to be squashed into just two commits before merge at least :p
|
90cdae9
to
5157ddd
Compare
Okay it's squashed :) |
Okay, so:
I think it's fine now. Thanks for patience and help, I learned a lot. |
pkgs/by-name/he/hellwal/package.nix
Outdated
platforms = [ | ||
"x86_64-linux" | ||
"x86_64-darwin" | ||
]; |
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 x86_64 specifically?
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.
Im pretty sure it failed previously before on aarch64, both linux and darwin, when I used platforms.all
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.
Okay it passed, I feel dumb.
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.
platforms.all
- hellwal: init at 1.0.2
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 not sure about platforms.all
either. Set it to platforms.unix
for now.
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: hellwal: init at 1.0.2
713eac3
to
705757d
Compare
705757d
to
64b5ea8
Compare
@ofborg build hellwal |
|
@pluiedev thanks for the kind and welcoming way you interacted in this PR. |
@pluiedev Thank you very much for helping me :) |
Hello!
I wanted to add hellwal here, it's color palette generator, something like pywal.
Here's a link to repo: hellwal
It's my first contribution here, so I don't really know If I did all correctly, please keep that in mind :)
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.