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

Remove deprecated pytest-runner dependency #547

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

penguinpee
Copy link

The pytest-runner project has been deprecated since 2019 and has been
archived on git hub in December 2024.

This removes the dependency on the deprecated package and also removes
deprecated keywords from setup.py related to it.

Use the new extra to install test dependencies in CI workflow.

@penguinpee
Copy link
Author

For the deprecation notice of pytest-runner see:

pytest-dev/pytest-runner@78a492c

For the removed/replaced deprecated keywords see:

https://setuptools.pypa.io/en/stable/references/keywords.html

@achilleas-k
Copy link
Member

We're going to have to install pytest on appveyor for the tests.

penguinpee and others added 2 commits March 29, 2025 11:27
The `pytest-runner` project has been deprecated since 2019 and has been
archived on git hub in December 2024.

This removes the dependency on the deprecated package and also removes
deprecated keywords from `setup.py` related to it.

Use the new extra to install test dependencies in CI workflow.
@jgrewe jgrewe force-pushed the drop_tests_require branch from 8664514 to 6e98933 Compare March 29, 2025 10:32
@coveralls
Copy link

coveralls commented Mar 29, 2025

Coverage Status

coverage: 78.615%. remained the same
when pulling e0ba5cc on penguinpee:drop_tests_require
into 5e5ec01 on G-Node:master.

@jgrewe jgrewe force-pushed the drop_tests_require branch from 689881d to 2b35bbe Compare March 29, 2025 10:54
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.

None yet

4 participants