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
pythonPackages.rdflib-jsonld: init at 0.5.0 #95711
Conversation
, rdflib | ||
}: | ||
buildPythonPackage rec { | ||
name = "rdflib-jsonld"; |
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.
name = "rdflib-jsonld"; | |
pname = "rdflib-jsonld"; |
url = "https://files.pythonhosted.org/packages/f5/17/45e137be0d93b70827fe5529c0400731344a978bc792193d7d9152e6dbe4/${name}-${version}.tar.gz"; | ||
sha256 = "4f7d55326405071c7bce9acf5484643bcb984eadb84a6503053367da207105ed"; | ||
}; | ||
doCheck = true; |
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.
default
doCheck = true; |
sha256 = "4f7d55326405071c7bce9acf5484643bcb984eadb84a6503053367da207105ed"; | ||
}; | ||
doCheck = true; | ||
format = "setuptools"; |
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.
default
format = "setuptools"; |
src = fetchurl { | ||
url = "https://files.pythonhosted.org/packages/f5/17/45e137be0d93b70827fe5529c0400731344a978bc792193d7d9152e6dbe4/${name}-${version}.tar.gz"; | ||
sha256 = "4f7d55326405071c7bce9acf5484643bcb984eadb84a6503053367da207105ed"; | ||
}; |
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.
use fetchPypi
src = fetchurl { | |
url = "https://files.pythonhosted.org/packages/f5/17/45e137be0d93b70827fe5529c0400731344a978bc792193d7d9152e6dbe4/${name}-${version}.tar.gz"; | |
sha256 = "4f7d55326405071c7bce9acf5484643bcb984eadb84a6503053367da207105ed"; | |
}; | |
src = fetchPypi { | |
inherit pname version; | |
sha256 = ...; | |
}; |
@koslambrou thanks for opening your first PR please look at https://github.com/NixOS/nixpkgs/blob/master/pkgs/development/python-modules/rope/default.nix for an example of a python package which uses nose |
i notcied you force pushed after closing the PR, did you mean to do that? |
@jonringer Oups not at all. Just finished creating the 2 commits with all the suggestions in my branch. |
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.
commits LGTM
has tests 👍
https://github.com/NixOS/nixpkgs/pull/95711
3 packages built:
python27Packages.rdflib-jsonld python37Packages.rdflib-jsonld python38Packages.rdflib-jsonld
sorry, another review below. Very close
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.
otherwise LGTM
https://github.com/NixOS/nixpkgs/pull/95711
3 packages built:
python27Packages.rdflib-jsonld python37Packages.rdflib-jsonld python38Packages.rdflib-jsonld
@jonringer I finished applying the changes. Thanks for the review :) |
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.
diff LGTM
commits LGTM
has tests 👍
https://github.com/NixOS/nixpkgs/pull/95711
3 packages built:
python27Packages.rdflib-jsonld python37Packages.rdflib-jsonld python38Packages.rdflib-jsonld
Motivation for this change
added rdflib-jsonld python package
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)