Read requirements from file in setup.py

[py-mine-bot]

Authored by Fyssion
Jan 20, 2021
Merged Jan 21, 2021

---

Hi,

I noticed the requirements in requirements.txt and the install_requires in setup.py don't match. This was causing some requirement issues when installing the lastest versions (5.0.0/5.1.0) from PyPI since asyncio-dgram is missing from the setup.py install_requires.

To fix this, I've changed setup.py to read from the requirements files.

[py-mine-bot]

Authored by kevinkjt2000
Jan 20, 2021

---

Thanks for doing this!
Unfortunately, pinning to exact versions in setup.py is generally discouraged. That TODO should have mentioned pinning to version ranges instead. Would you mind updating the requirements.txt file to have >= symbols instead of ==? I can merge after that.

[py-mine-bot]

Authored by Fyssion
Jan 20, 2021

---

Alright, I think I've done that 👍