search

mosquito-cr / mosquito

A background task runner for crystal applications supporting periodic (CRON) and manually queued jobs
MIT License
227 stars 24 forks source link

zrange redis command fix + more permissive redis shard version #125

Closed rmarronnier closed 5 months ago

rmarronnier commented 10 months ago

The redis shard version requirements of https://github.com/defense-cr/defense conflicts with the requirements of the mosquito shard.

While trying the tests with the last redis shard version 0.8.0, the new changes in https://github.com/jgaskins/redis/commit/70266be498b1d448594b5ab2eb098600dd70dbb8 triggered failures.

I've included a suggested fix with this PR.

Feel free to modify / comment for a better solution.

robacarp commented 10 months ago

@rmarronnier thank you, this is good as is.