nix-community / nixpkgs-wayland

Automated, pre-built packages for Wayland (sway/wlroots) tools for NixOS. [maintainers=@colemickens, @Artturin]
496 stars 44 forks source link

Add wbg #424

Closed SrEstegosaurio closed 9 months ago

SrEstegosaurio commented 10 months ago

Add wbg to the list of derivations.

wbg is a small utility to set a wallpaper image written in C.

P.S: I'm very new to contributing so I'm sorry if I done something dumb or just wasted your time. Let me know if that's the case.

colemickens commented 10 months ago

Did you maybe re-format the tree? I see some other edits in there. Not a huge deal, just curious.

SrEstegosaurio commented 10 months ago

I did ran a nix fmt. It seemed like the correct thing to do. I thought it only affected my code. 🤔

Artturin commented 10 months ago

The formatting changes can stay, we should make the CI run nix fmt automatically.

Artturin commented 9 months ago

Committed in https://github.com/nix-community/nixpkgs-wayland/commit/edba7d689bc9559398d71fa531c052074bc86ef3

Had to do it that way instead of editing your PR as maintainers cannot push to the PR's branch when master is used

I recommend reading https://github.com/NixOS/nixpkgs/blob/master/CONTRIBUTING.md#how-to-create-pull-requests

SrEstegosaurio commented 9 months ago

Sorry for not having read it beforehand. Thanks for pointing me to the documentation.