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.googletrans: init at 2.4.0 #80430
Conversation
@ofborg build python37Packages.googletrans |
6753276
to
a71e107
Compare
@ofborg build python37Packages.googletrans |
Hum... now tests run but I can see these becoming flaky if too many builds are triggered and Also, all green but both builds report failure. I'm a bit confused. I feel tempted to disable tests and add a comment about this. Advice? |
tests occur in a sandbox (they don't have access to network), you don't have to worry about it pinging any service |
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.
running the tests isn't really meaningful as they try to ping endpoint just apply the following
Add suggestions included. |
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
commit LGTM
[5 built, 0.0 MiB DL]
https://github.com/NixOS/nixpkgs/pull/80430
3 package built:
python27Packages.googletrans python37Packages.googletrans python38Packages.googletrans
@GrahamcOfBorg build python27Packages.googletrans python37Packages.googletrans python38Packages.googletrans |
Motivation for this change
Needed for
mnemosyne
- see: #80324Things 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)