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-runner setuptools integration #245

Closed
abravalheri opened this issue Jul 28, 2019 · 5 comments
Closed

Remove pytest-runner setuptools integration #245

abravalheri opened this issue Jul 28, 2019 · 5 comments
Labels
enhancement New feature or request
Milestone

Comments

@abravalheri
Copy link
Collaborator

Recently pytest-dev/pytest#5546 was merged with a discussion about not recommending pytest-runner integration with setuptools.

Does this affect PyScaffold in any sense?

@abravalheri abravalheri added the question Further information is requested label Jul 28, 2019
@FlorianWilhelm
Copy link
Member

@abravalheri, good point. Personally, I don't use python setup.py test at all anymore but run py.test or tox directly. If this is now recommended I would also be in favour of just dropping the runner. Would doctest still work? Would we lose anything else? We should then also remove it from the documentation.

@abravalheri
Copy link
Collaborator Author

I am under the impression that we would need to create a explicit tox testenv for doctest.
Something like this: https://ofosos.org/2019/01/06/doctest-travis/

I think that is the main drawback, isn't it? I don't think we have anything extra to loose...

@FlorianWilhelm
Copy link
Member

Agreed. Judging from the source code of ptr, it's not responsible for the doctest command of setup.py but it's coming from Sphinx. So we could just remove ptr and state it in the Changelog.rst as well that we dropped it. A unit test that doctest still works would be nice but not really necessary since Sphinx registers this and we are in no way messing with this.

@FlorianWilhelm FlorianWilhelm added this to the v4.0 milestone Sep 22, 2019
@jayvdb
Copy link
Contributor

jayvdb commented Oct 23, 2019

setuptools has also deprecated test. pypa/setuptools#1684

@FlorianWilhelm
Copy link
Member

Okay, then the decision is made. Let's remove it in version 4.0 from the docs and als the py.test runner contrib package. I am quite happy with this since it will reduce the code and complexity. I haven't used python setup.py test for ages and always call py.test directly.

Any volunteers to provide a PR for the v4.0.x branch?

@FlorianWilhelm FlorianWilhelm added enhancement New feature or request and removed question Further information is requested labels Oct 24, 2019
@FlorianWilhelm FlorianWilhelm changed the title It seems that pytest is moving away from recommending pytest-runner setuptools integration Remove pytest-runner setuptools integration Oct 24, 2019
abravalheri added a commit to abravalheri/pyscaffold that referenced this issue Jul 6, 2020
FlorianWilhelm pushed a commit that referenced this issue Jul 10, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

3 participants