setup_requires -> tests_requires, for pytest

fugue / credstash

A little utility for managing credentials in the cloud

Apache License 2.0

2.06k stars 215 forks source link

setup_requires -> tests_requires, for pytest #296

Open jonringer opened 4 years ago

jonringer commented 4 years ago

pytest>=5 isn't compatible with python2 https://github.com/pytest-dev/pytest/blob/eaf7ce9a992cea7506553b7d77fdf9622900ae36/setup.cfg#L43

This makes the python2 version fail, even though it may be compatible, as noted: https://github.com/NixOS/nixpkgs/pull/92561#pullrequestreview-459470792

For your CI, it looks like this is irrelevant because your tox.ini will install it anyway, and the python setup.py test command is deprecated anyway.

jonringer commented 3 years ago

Is this is still relevant? sorry, i get ~100 notifications a day and this got buried