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

Question

HumanTaikan opened this issue 3 years ago · 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

Answer 1 · 2021-08-18T20:04:36.000Z

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

Answer 2 · 2021-08-18T20:12:38.000Z

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

Answer 3 · 2021-08-18T20:25:56.000Z

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

Answer 4 · 2021-08-18T20:41:34.000Z

i also pushed 2.8.2 which should fix it.

Answer 5 · 2021-08-18T20:45:50.000Z

It works, thanks a lot and have a nice day