stefanwille/crystal-redis

Error: method Redis::TransactionApi#zadd must return (Int64 | String) but it is returning Redis::Future

HumanTaikan opened this issue · 5 comments

Hello,

I'm doing some testing with crystal and sidekick.cr and it looks like I got an issue with redis (error in title).

I join the log trace here:
trace.log

Crystal 1.1.1
dependencies:
redis:
github: stefanwille/crystal-redis
version: ~> 2.6

Reproduce:
clone repo: https://github.com/mperham/sidekiq.cr
shards install
crystal run examples/sidekiq.cr

Issue opened on sidekick: hugopl/sidekiq.cr#111

It works just adding the type to the method, I don't know yet what it implies but I let it there:
commands.txt

if you lock crystal-redis version at 2.7.0 would it work?

i also pushed 2.8.2 which should fix it.

It works, thanks a lot and have a nice day