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: 0.6.0 -> 0.7.0 #34768
Conversation
# install_requires | ||
dateutil pytz regex tzlocal | ||
# extra_requires | ||
(if isPy3k then null else umalqurra) |
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.
++ lib.optional isPy3k umalqurra
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.
I had that in the first place, but I want to somehow make clear it is an extra requirement.
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.
Why would it be clearer this way? Simply stating that it is optional (extra_requires) is sufficient.
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.
One could think it is required for Python 2. But I'll change it anyway.
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.
Actually, it seems like umalqurra is now supported on Python 3. No need for lib.optional.
I have successfully run |
Thanks for your work! |
@makefu I hope to meet you at Shackspace on Wednesday |
@dotlambda sure do! |
Motivation for this change
YAML was replaced by ruamel: scrapinghub/dateparser@fee300c
Also did some further refactoring.
Things done
build-use-sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)/cc @makefu