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
python3Packages.pylibgen: 2.0.1 -> 2.0.2 #66242
Conversation
@GrahamcOfBorg build papis |
@@ -19,7 +21,7 @@ buildPythonPackage rec { | |||
|
|||
checkInputs = [ pytest ]; | |||
|
|||
# no tests in PyPI tarball | |||
# too lazy to add pre-commit package |
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.
what does this mean, we have gitAndTools.pre-commit
maybe you can use toPythonModule
?
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.
there's a seperate pypi package with the same name. I was trying to go home and just wanted to push the progress. I guess i didn't have to open a pr... xD
I'm moving from nix to guix, so if somebody wants to take the ownership of the package s/he's welcome |
@nico202 unrelated to the PR, but I'm curious as to the motive behind moving to guix. Other than being able to use scheme and more of a focus on libre software, I'm not sure of a big difference. |
Well the focus on the libre software is an important issue for me. Also, if at first I had difficulties in writing packages in scheme, now I find it to be a better language (after years I was able to have julia packages almost working, I was not able to do the same in nix). Another thing that at first I thought was better in nix was nix-shell, but guix environment is way more powerful. Then, I don't think guix pack (create distributable tarballs) has something similar in the nix world. Lastly, being an emacs user, the emacs-guix interface is superior to nox or anything else. I'm really happy with guix, even if there are way less packages, package management is a slower and to send patches you need emails :D |
@nico202 thanks for the response. Wish you the best :) |
Were you going to address #66242 (comment) @jonringer? |
yea, sorry, Keep getting distracted ;) |
8c3a5f5
to
5dc20b6
Compare
@worldofpeace looks llike all the tests try to do a network call..... so it was all for not, trying to get tests working 🤣 |
Aww. Sorry about that. |
you couldn't have know, I couldn't have known :( |
d955853
to
7e67bcd
Compare
@GrahamcOfBorg build pythonPackages.pre-commit python3Packages.pre-commit python36Packages.pylibgen |
moved the pre-commit package from being an application under git-and-tools, to a proper python packages (supports python{2.7,3.5,3.6,3.7} now), then used the "python3Packages.toPythonApplication" function to import it back under the git-and-tools namespace. Also added an alias for it under all-packages to the gitAndTools.pre-commit package. pylibgen now builds with python3.{6,7} |
7e67bcd
to
4f3b345
Compare
Add pkgs.gitAndTools.pre-commit alias Add pkgs.pre-commit alias
4f3b345
to
cbb4089
Compare
@FRidh rebased off master, no more merge conflicts. Edited pre-commit message to now reflect that i moved the expression from gitAndTools to pythonPackages, |
Motivation for this change
noticed it was broken while reviewing one of my other PRs
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)Notify maintainers
cc @nico202