search

tobymao / saq

Simple Async Queues
https://saq-py.readthedocs.io/en/latest/
MIT License
586 stars 41 forks source link

Relax redis dependency #61

Closed gazorby closed 1 year ago

gazorby commented 1 year ago

Hi @tobymao, great project!

redis dependency is specified as >4.2,<4.4. Any reason why we need <4.4? Not a big deal but the readme only mention >4.2

pahrohfit commented 1 year ago

It was failing in a few areas, as well as tossing up lots of uncaught exceptions. I just submitted a PR(#62) a few minutes ago which includes fixes and relaxed dependency (<5.0).

tobymao commented 1 year ago

thanks yea, redis 4.4 broke a lot of projects with their changes to errors / cancellations.

tobymao commented 1 year ago

https://github.com/redis/redis-py/pull/2104