-
-
Notifications
You must be signed in to change notification settings - Fork 282
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 is an unnecessary dependency? #2068
Comments
we need pytest because of this class |
Thanks. I see that the zarr-python/src/zarr/testing/__init__.py Lines 4 to 7 in 064b2e0
Here's the behavior I see in an environment without pytest:
|
This seems right! (@jhamman feel free to issue a correction here). |
Pytest should not be a required dependency so +1 to removing it. |
Zarr version
main
Numcodecs version
na
Python Version
na
Operating System
na
Installation
na
Description
zarr-python/pyproject.toml
Line 35 in 064b2e0
Looks like that was added in #1785. It wasn't discussed, so it was presumably intended to be a dev-only dependency.
Slightly related, but would you be open to defining
tool.hath.envs.test
as a standard optional dependency?zarr-python/pyproject.toml
Line 106 in 064b2e0
which should work for both.
Steps to reproduce
na
Additional output
No response
The text was updated successfully, but these errors were encountered: