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 pytest from setup requirements #147

Closed
wants to merge 1 commit into from

Conversation

Singletoned
Copy link

It's not a great package to have in production, and doesn't actually seem to be needed for anything apart from running tests.

@liZe
Copy link
Member

liZe commented Feb 13, 2020

Hi!

Thank you for this pull request.

Pytest-runner is not Pytest, and it’s required in setup_requires if we want to have setup.py test work without installing pytest before.

Having pytest-runner in production is bad. Having pytest-runner is actually now bad everywhere, as it’s deprecated. But I didn’t find a clean solution yet to launch tests out of the box.

I’ll change CairoCFFI’s (and other projects’) packaging during the next months, and this change will include the removal of pytest-runner.

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

2 participants