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
sphinx-autobuild: init at 2020.9.1 #109565
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.
You need to add an entry to python top level file.
8522620
to
e482213
Compare
I added it to |
This is a semi-automatic executed nixpkgs-review which does not build all packages (e.g. lumo, tensorflow or pytorch) Result of 3 packages built:
|
1c989a8
to
38f9cd1
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.
other then that diff LGTM
This is a semi-automatic executed nixpkgs-review which is checked by a human on a best effort basis and does not build all packages (e.g. lumo, tensorflow or pytorch). Result of 3 packages built:
|
Please do not add fixup commits. GitHub does not always resolve them correctly. |
Python command line tool for autobuilding sphinx documentations. Co-authored-by: Sandro <sandro.jaeckel@gmail.com> Co-authored-by: Fabian Affolter <mail@fabian-affolter.ch>
Addresses #109564
Motivation for this change
Package seems to be missing albeit useful.
Be mindful, this is my first contribution to
nixpkgs
. I build the package locally in my shell.nix setup but it isn't exactly straightforward for me to test my patch innixpkgs
for some reason. I tried to produce a diff that looks like other Python package definitions.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)