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.dateparser: switch to parameterized and thereby fix Hydra test #34643
Conversation
nose-parameterized is deprecated and yields a warning
Seems like a good solution to me. While you're at it, could you also remove |
In the same commit? |
Removed |
@FRidh see my comment in the other PR, shouldn't we rather drop the |
I grepped through |
Let's just get rid of this deprecated package as it simplifies things. |
Thanks for your clarification, then you're right :)
I wasn't sure at first if we should really drop this package, however I think that you might be right. EDIT: @dotlambda was faster, thank you :-D |
I already made a PR: scrapinghub/dateparser#381 |
I ran |
awesome! Thanks for caring about this <3 |
@FRidh Please merge. |
Motivation for this change
nose-parameterized is deprecated and yields a warning.
This fixes a failing Hydra test: https://hydra.nixos.org/build/68773666
The relevant upstream PR (scrapinghub/dateparser#381) can't be used for
fetchpatch
. The codebase seems to have changed too much in the meantime.See also #34589.
Things done
build-use-sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)/cc @Ma27