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
mopidy-scrobbler: init at 2.0.1 #107993
mopidy-scrobbler: init at 2.0.1 #107993
Conversation
Result of 1 package built:
|
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.
Other than the sorting issue this is AOK
ef579c4
to
9ae22d6
Compare
This is a semi-automatic executed nixpkgs-review. If you find some bugs or got suggestions for further things to search or run please reach out to SuperSandro2000 on IRC. Result of 1 package built:
|
Result of 1 package built:
|
Sort jakeisnt in maintainer list maintainers: add jakeisnt
edc39e0
to
48cd27a
Compare
283e8d9
to
476ada5
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.
Would you mind to sort the packages in a commit before adding the new package? This makes this easier to review and keep the changes on individual and logical commits.
Sure. What would you like the commit message to be? Would you prefer the commit to sort the packages to be before or after the commit to add mopidy-scrobbler? |
@jakeisnt I'd probably do it before. |
Result of 1 package built:
|
This is a semi-automatic executed nixpkgs-review. If you find some bugs or got suggestions for further things to search or run please reach out to SuperSandro2000 on IRC. Result of 1 package built:
|
476ada5
to
a7de5ab
Compare
Please change |
a7de5ab
to
c961da0
Compare
c961da0
to
c392041
Compare
This has been done. |
This is a semi-automatic executed nixpkgs-review. If you find some bugs or got suggestions for further things to search or run please reach out to SuperSandro2000 on IRC. Result of 1 package built:
|
Motivation for this change
Mopidy-Scrobbler is a Mopidy extensions maintained by the Mopidy organization. Many other Mopidy extensions maintained by them are already packaged.
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)