Skip to content
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

upgrade "convertdate" to >= 2.3.0 #406

Closed
mehditerraneen opened this issue Dec 8, 2020 · 14 comments
Closed

upgrade "convertdate" to >= 2.3.0 #406

mehditerraneen opened this issue Dec 8, 2020 · 14 comments

Comments

@mehditerraneen
Copy link

Hi, Is it possible to check if upgrading to convertdate version >=2.3.0 does not have a big impact ?
Thanks

@MartinLinden
Copy link

Hi! Great package, thanks for that. I would very much support an update on convertdate. The current version limits pytz to be < 2020 which makes the library run into dependency problems with other libraries

@zhaoyongjie
Copy link

Awesome library for holiday searching. but current pytz < 2020 conflict with many current popular libraries.

@fabaff
Copy link
Contributor

fabaff commented Dec 14, 2020

convertdate<=2.2.0 is an issue for the Fedora package.

Reference: https://bugzilla.redhat.com/show_bug.cgi?id=1905176

@ondrejj
Copy link

ondrejj commented Dec 28, 2020

According to convertdate changes (https://github.com/fitnr/convertdate/releases/tag/v2.3.0) there are no changes, which should affect holidays.
Why there is version requirement for convertdate<=2.2?

@lociii
Copy link

lociii commented Jan 15, 2021

Limitation introduced in b57cb2d by @mmontel-wgs and released in 0.10.4 - 0.10.3 is not affected.
Maybe he can give an explanation.

@dr-prodigy
Copy link
Collaborator

dr-prodigy commented Jan 31, 2021

The limitation has in fact been introduced due to convertdate 2.2.0 breaking changes (most notable ones: support for Python 2 dropped, some method signature changes, etc.), which prevented us to accept it immediately.
I decided (mmontel-wgs is my 2nd account) to release a last Python 2 compatible version (0.10.4), to remove this limit in the upcoming, now in beta, version. This of course required a "keep-back" for convertdate.

Pls refer to https://github.com/dr-prodigy/python-holidays#development-version for how to change your holidays lib to beta version until it gets finalized and released in master, thx.

@lociii
Copy link

lociii commented Jan 31, 2021

@dr-prodigy Thanks for the explanation. Do you have any ETA on the next release?

@dr-prodigy
Copy link
Collaborator

hi @lociii in fact, no, I don't have set a date yet: usually I sum up a reasonable number of updates before releasing: in the meantime, for who needs bleeding edge updates, the beta branch is the way to go (no untested / unstable features on it, never).
https://github.com/dr-prodigy/python-holidays#development-version
Does this suit your needs?
Pls let me know, thx

@dr-prodigy dr-prodigy reopened this Feb 1, 2021
@lociii
Copy link

lociii commented Feb 3, 2021

That might work for an end user but not for a package depending on python-holidays.
I'd deeply recommend releasing a new version even if you haven't accumulated too many changes as the pytz < 2020 might be blocking some important updates on other packages.

@mweinelt
Copy link

mweinelt commented Feb 3, 2021

We are having the same problem in NixOS packaging sadly. Please release a version supporting recent convertdate versions.

@dr-prodigy
Copy link
Collaborator

Hi,
I confirm I've just released v0.10.5 on master and pypi to solve the issues: please keep me posted on your status.
Thank you all for your interest and support.

@mweinelt
Copy link

mweinelt commented Feb 3, 2021

Thanks! One issue came up: hijri_converter is missing in the list of dependencies.

@dr-prodigy
Copy link
Collaborator

dr-prodigy commented Feb 3, 2021

Hi,
it should be fixed now and released as 0.10.5.2 [EDIT: was 0.10.5.1] on pypi.
Pls let me know, thank you very much!

@dr-prodigy
Copy link
Collaborator

Confirmed as fixed on other similar issue. Closing, thx 👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

8 participants