-
-
Notifications
You must be signed in to change notification settings - Fork 13.9k
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
gaphor: 2.8.2 -> 2.25.1; better-exceptions: init at 0.3.3 #322887
base: master
Are you sure you want to change the base?
Conversation
/marvin opt-in |
|
||
src = fetchPypi { | ||
inherit pname version; | ||
sha256 = "sha256-5Oa8GERNXwTm6JSxA4Hl6SHT1UQkBBgWLH21fp6zRTs="; |
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.
At present, it is not recommended to use sha256 as a parameter for fetcher, but to use the hash of SRI mode as a parameter.
You could use following command to generate SRI hash.
nix hash to-sri --type sha256 $(nix-prefetch-url )
sha256 = "sha256-5Oa8GERNXwTm6JSxA4Hl6SHT1UQkBBgWLH21fp6zRTs="; | |
hash = "sha256- ... ="; |
meta = with lib; { | ||
description = "Pretty and helpful exceptions, automatically"; | ||
homepage = "https://github.com/qix-/better-exceptions"; | ||
license = licenses.mit; | ||
maintainers = [ ]; | ||
}; |
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.
Currently, use with lib in meta is not recommanded anymore.
Consider to add yourself as package maintainers.
I not sure if package from pypi should add binaryNativeCode. You may consider if it should be.
meta = with lib; { | |
description = "Pretty and helpful exceptions, automatically"; | |
homepage = "https://github.com/qix-/better-exceptions"; | |
license = licenses.mit; | |
maintainers = [ ]; | |
}; | |
meta = { | |
description = "Pretty and helpful exceptions, automatically"; | |
homepage = "https://github.com/qix-/better-exceptions"; | |
license = lib.licenses.mit; | |
sourceProvenance = with lib.sourceTypes; [ binaryNativeCode ]; | |
maintainers = with lib.maintainers; [ ]; | |
}; |
|
||
format = "pyproject"; | ||
|
||
src = fetchPypi { | ||
inherit pname version; | ||
sha256 = "sha256-+qqsSLjdY2I19fxdfkOEQ9DhTTHccUDll4O5yqtLiz0="; | ||
sha256 = "sha256-9PNgU1/9RL6QXba0gn4zHCAtSV3iY0HOY1Rf6bkyzxY="; |
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.
same as previous
Motivation for this change
#291281
Description of changes
Bumped version number and added/updated dependencies as needed.
Added better-exceptions as a new package because it is a dependency of gaphor.
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.