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
rofi-mpd: init at 1.1.0 #69877
rofi-mpd: init at 1.1.0 #69877
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.
nix-review
passes on NixOS
diff LGTM (Good job! :) )
executable seems to work (wasn't able to test it fully, but author I'm sure did this :) )
leaf package
please just squash the commits into 2:
maintainers: add jakestanger
rofi-mpd: init at 1.1.0
@GrahamcOfBorg build rofi-mpd |
not sure where your git-fu is right now, but for squashing I would do:
see more info on contributing guideliness here |
05a287a
to
f9ea4bc
Compare
My git-fu definitely wasn't good enough to do that without your help. Thanks a lot :) |
@GrahamcOfBorg build rofi-mpd |
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.
Sorry, just now noticed these :(.
after you apply the changes I would do:
git status # ensure you only have relevant changes
git add .
git commit --amend --no-edit
git push jakestanger rofi-mpd --force
f9ea4bc
to
dc87dba
Compare
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.
nix-review
passes on NixOS
diff LGTM
executable seems to work
leaf package
[2 built, 1 copied (1.2 MiB), 0.3 MiB DL]
https://github.com/NixOS/nixpkgs/pull/69877
1 package were build:
rofi-mpd
I'll allow for some other reviewers to comment, but it all LGTM.
Thanks for bearing through all the reviews/corrections :)
@GrahamcOfBorg build rofi-mpd |
dc87dba
to
499deed
Compare
builds on linux and darwin
|
Motivation for this change
Adds package.
Things done
sandbox
innix.conf
on non-NixOS)nix-shell -p nix-review --run "nix-review wip"
./result/bin/
)nix path-info -S
before and after)