search

rmax / scrapy-redis

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

Cleanup requirements #268

Closed devfox-se closed 1 year ago

devfox-se commented 1 year ago

Currently looking into the requirements that come along the package

https://github.com/rmax/scrapy-redis/blob/master/requirements.txt

scrapy>=2.0
redis>=4.0
six>=1.15
coverage
flake8
mock
pytest
tox
pytest-cov

these packages are completly uncecessary to perform the main package. I believe it is a good idea to get rid of them to not make the projects that rely on scrapy-redis filled with unnecessary packages

coverage
flake8
mock
pytest
tox
pytest-cov
rmax commented 1 year ago

You are right, there is a mix with requirements, ideally these should go to requirements-test.txt.