search

stefanwille / crystal-redis

Full featured Redis client for Crystal
MIT License
381 stars 61 forks source link

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

Closed HumanTaikan closed 2 years ago

HumanTaikan commented 2 years ago

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

HumanTaikan commented 2 years ago

Issue opened on sidekick: https://github.com/mperham/sidekiq.cr/issues/111

HumanTaikan commented 2 years ago

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

kostya commented 2 years ago

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

kostya commented 2 years ago

i also pushed 2.8.2 which should fix it.

HumanTaikan commented 2 years ago

It works, thanks a lot and have a nice day