Skip to content
This repository has been archived by the owner on Jan 2, 2024. It is now read-only.

Remove pytest dependency? #54

Closed
spookylukey opened this issue Aug 19, 2022 · 1 comment · Fixed by #55
Closed

Remove pytest dependency? #54

spookylukey opened this issue Aug 19, 2022 · 1 comment · Fixed by #55

Comments

@spookylukey
Copy link

Python Version

No response

pytest Version

No response

Package Version

No response

Description

Does this need to have an install_requires dependency on pytest?

Specifically, this means that installing pytest-is-running means you get pytest installed, even if you don't need that (e.g. in production). I would have thought that, given the implementation and the intended usage, that we should be able to avoid this hard dependency.

@adamchainz
Copy link
Owner

Very good point! Fixed in #55 and released in 1.3.0.

Btw I've been considering dropping this package and only recommending "pytest" in sys.modules, which is just as good for most use cases.

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

Successfully merging a pull request may close this issue.

2 participants