splitrb / split

:chart_with_upwards_trend: The Rack Based A/B testing framework
https://rubygems.org/gems/split
MIT License
2.71k stars 368 forks source link

Fix deprecation warning with Redis 4.8.0 #701

Closed martingregoire closed 1 year ago

martingregoire commented 1 year ago

Fixes the following deprecation warning when using Redis 4.8.0:

Redis#sadd will always return an Integer in Redis 5.0.0.
Use Redis#sadd? instead.

As the return value of redis.sadd is not used, we can simply replace the .sadd call with .sadd?.

see