-
Notifications
You must be signed in to change notification settings - Fork 27
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
Replace distutils.version with packaging.version's parse #36
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.
Thanks!
Any idea when we can see this released? Work around for me is currently manually changing the code in my venv. |
I'm seeing this too now. Any updates? |
@ktosiek @thijskramer @pquentin is there anything we can do to speed up the merging of this PR? |
Patch supplied upstream [0] but the maintainer hasn't merged it and there are signs of the project being abandoned. This is a temporary fix until the future of the project is resolved. [0] ktosiek/pytest-freezegun#36
Patch supplied upstream [0] but the maintainer hasn't merged it and there are signs of the project being abandoned. This is a temporary fix until the future of the project is resolved. [0] ktosiek/pytest-freezegun#36
Patch supplied upstream [0] but the maintainer hasn't merged it and there are signs of the project being abandoned. This is a temporary fix until the future of the project is resolved. [0] ktosiek/pytest-freezegun#36
Joining the question.
|
Honestly, at this point I do wonder if there's really a point in supporting pytest < 3.6.0, and if it wouldn't be better to just drop that and avoid the extra dep. |
Good idea, please see #38 as an alternative. |
Closing this year old PR, happy to re-open if there's any interest, but will keep #38 open a bit longer. |
@ktosiek would you try to recruit more maintainers? |
Fixes #35.
Demo