rmax / scrapy-redis

Redis-based components for Scrapy.
http://scrapy-redis.readthedocs.io
MIT License
5.54k stars 1.59k forks source link

doc: fix redis-py dependency version #199

Closed tsonglew closed 3 years ago

tsonglew commented 3 years ago

In redis-py < 3.0, redis.Redis.spop() takes only two arguments which are self and name. But self.fetch_data(self.redis_key, self.redis_batch_size) applies self and two other arguments to self.fetch_data which is redis.Redis.spop() indeed. This could cause an error TypeError: spop() takes 2 positional arguments but 3 were given