-
-
Notifications
You must be signed in to change notification settings - Fork 14.7k
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
spotify-cli-linux: 1.9.0 -> 1.9.1 #372411
base: master
Are you sure you want to change the base?
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.
Also please change the commit message and pull request title according to the contributing guide.
src = fetchFromGitHub { | ||
owner = "pwittchen"; | ||
repo = "spotify-cli-linux"; | ||
rev = "v${version}"; |
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.
rev = "v${version}"; | |
tag = "v${version}"; |
7ce83b9
to
1f51cab
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.
nitpick: Spotify-cli-linux
in the commit message and pr title should be lowercase to match the package name
Also changed fetchPypi to fetchFromGitHub
|
Update from 1.9.0 to 1.9.1 and change from fetchPypi to fetchFromGitHub.
Changelog for spotify-cli-linux:
https://github.com/pwittchen/spotify-cli-linux/releases/tag/v1.9.1
This seems to fix #372260.
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.