Skip to content

[iss623] Move pytest to development dependencies#630

Open
Dodothereal wants to merge 1 commit into
brightwind-dev:masterfrom
Dodothereal:fix/623-pytest-dependencies-pytest-is-declared-a
Open

[iss623] Move pytest to development dependencies#630
Dodothereal wants to merge 1 commit into
brightwind-dev:masterfrom
Dodothereal:fix/623-pytest-dependencies-pytest-is-declared-a

Conversation

@Dodothereal

Copy link
Copy Markdown

[iss623] Move pytest to development dependencies

Summary

pytest was declared as a runtime dependency in both package metadata and the requirements file, even though it is only used by the test suite. This removes it from normal installations and makes it available through the dev extra for contributors.

Fixes #623

Changes

  • Remove pytest from install_requires and requirements.txt.
  • Add pytest>=4.1.0 to the dev optional dependency extra.

Testing

Ran in the required sandbox:

python -m venv /tmp/brightwind-runtime && /tmp/brightwind-runtime/bin/python -m pip install --upgrade pip && /tmp/brightwind-runtime/bin/python -m pip install . && /tmp/brightwind-runtime/bin/python -c "import importlib.util; assert importlib.util.find_spec('pytest') is None" && /tmp/brightwind-runtime/bin/python -m pip install '.[dev]' && /tmp/brightwind-runtime/bin/python -c "import pytest; print(pytest.__version__)"

The normal package installation completed without pytest; installing .[dev] then installed and imported pytest successfully.


AI assistance disclosure

This contribution was produced by an autonomous AI coding agent (Claude Code) that @Dodothereal operates and monitors. @Dodothereal is accountable for it, will address review feedback promptly, and will close this PR immediately if this kind of contribution is unwelcome in this project. Commits carry an Assisted-by: Claude Code trailer.

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

Successfully merging this pull request may close these issues.

[pytest/dependencies] pytest is declared as a runtime install dependency

1 participant