Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

pytest should't be in install_requires in seup.py #246

Open
yurivict opened this issue Jun 11, 2023 · 1 comment
Open

pytest should't be in install_requires in seup.py #246

yurivict opened this issue Jun 11, 2023 · 1 comment

Comments

@yurivict
Copy link

pytest is only required for testing, and not for running the module.

@ptpan
Copy link
Contributor

ptpan commented Dec 7, 2023

Hi @yurivict,

Sorry for getting back to you so late. I think you are correct that pytest is used for testing and not for running the PyMTL package. However we do have a set of testing methodologies that build on pytest (and hypothesis). There are many use cases of that methodology and we think it makes sense to include pytest and hypothesis by default.

Does having pytest/hypothesis as a dependency complicate your use case? I think so far our experiences has been pretty smooth, and pip install does pretty good jobs handling the installation..

Peitian

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants