-
Notifications
You must be signed in to change notification settings - Fork 612
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
Update six version requirement to >= 1.10, < 2.0 #279
base: master
Are you sure you want to change the base?
Conversation
Closed by #292. |
@jpweytjens #292 pins six to 1.12 rather than making it >= 1.12 and < 2.0 as proposed here. Are there any known issues with using newer versions? Is it feasible to switch to a range? |
I'm interested in that as well. I'm not familiar with six, but if we could allow later releases (like >= 1.14) as well would help a lot. I would welcome to switch to the range mentioned by @mikebridge rather tan pinning it to a fixed version, if this is feasilbe. |
@jpweytjens This is still an issue today is conflicting with newer versions of other packages |
@traverseda What's your opinion? I've heard arguments both ways with respect to pinning fixed versions, setting a range or keeping a dependency up to date. |
Any updates on whether or not it's possible to use a later release of six? |
These outdated fixed version dependencies really mess up installations and cause conflicts unnecessarily in an unparalleled manner - hardly seen elsewhere. |
I totally hear you on that. Thanks for the feedback.
I obviously have not had too much time to maintain this package in a while.
Would someone like to put together a PR to address this issue?
…On Sat, Mar 12, 2022, 07:24 kxrob ***@***.***> wrote:
These outdated fixed version dependencies really mess up installations and
cause conflicts unnecessarily in an unparalleled manner - hardly seen
elsewhere.
Better use >= or nothing (and < only by exception WHEN concrete knowledge
of problems which cannot be fixed easily.)
—
Reply to this email directly, view it on GitHub
<#279 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAB6NODIFYR47VJBP2PFUEDU7SLHHANCNFSM4HHY6BBA>
.
You are receiving this because you are subscribed to this thread.Message
ID: ***@***.***>
|
I just tested upgrading six to 1.16.0 and there seemed to be no issues running textract! |
Current version of six is 1.12. Being conservative by keeping less than 2.0.