search

deanmalmgren / textract

extract text from any document. no muss. no fuss.
http://textract.readthedocs.io
MIT License
3.89k stars 599 forks source link

Update six version requirement to >= 1.10, < 2.0 #279

Open DanielSwain opened 5 years ago

DanielSwain commented 5 years ago

Current version of six is 1.12. Being conservative by keeping less than 2.0.

jpweytjens commented 5 years ago

Closed by #292.

mikebridge commented 4 years ago

@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?

ellr commented 3 years ago

@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.

vladox commented 3 years ago

@jpweytjens This is still an issue today is conflicting with newer versions of other packages

jpweytjens commented 3 years ago

@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.

makretch commented 2 years ago

Any updates on whether or not it's possible to use a later release of six?

kxrob commented 2 years ago

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.)

deanmalmgren commented 2 years ago

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 https://github.com/deanmalmgren/textract/pull/279#issuecomment-1065883018, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAB6NODIFYR47VJBP2PFUEDU7SLHHANCNFSM4HHY6BBA . You are receiving this because you are subscribed to this thread.Message ID: @.***>

jaluoma commented 1 week ago

I just tested upgrading six to 1.16.0 and there seemed to be no issues running textract!