bear / parsedatetime

Parse human-readable date/time strings
Apache License 2.0
694 stars 107 forks source link

Abuse of `setup_requires` for the non-runtime dependency `pytest-runner` #219

Closed ankostis closed 7 years ago

ankostis commented 7 years ago

The setup.py script has as a setup requirement a test-time one, pytest-runner. According to the docs, setup_requires is for dependencies of the setup-tools themselves.

Such requirements typically are placed in the tests_require argument. If this is currently impossible due to some installation script, there is additionally the test_suite argument for facilitating the launching of tests.

Thank you for your time.

bear commented 7 years ago

PR has been merged