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

Testing requirements are outdated #602

Open
aqeelat opened this issue Jul 9, 2023 · 3 comments
Open

Testing requirements are outdated #602

aqeelat opened this issue Jul 9, 2023 · 3 comments

Comments

@aqeelat
Copy link

aqeelat commented Jul 9, 2023

I'm going through the poetry.lock file in one of my projects and searching for packages that still use six in order to help the package maintainers drop support for python 2.7.

However, pytest-cov lists six as a testing requirement even though it does not support python 2.7. I also came across a couple of issues that I would like to let you know about.

setup.py says that:

  1. six is required but it is not actually used in the project
  2. process-tests is in setup.py but not in ci/requirements.txt
  3. hunter was added in 0d75fd2 with no code dependency. I think this might be a local debugging tool that @ionelmc was using.

pytest-cov/setup.py

Lines 136 to 145 in 2c9f217

extras_require={
'testing': [
'fields',
'hunter',
'process-tests',
'six',
'pytest-xdist',
'virtualenv',
]
},

These dependencies were moved from tox which made them hard to track.

@RonnyPfannschmidt
Copy link
Member

Thanks for digging those oversights up,now they can be resolved

The quickest way may be a PR with the fix

@aqeelat
Copy link
Author

aqeelat commented Jul 9, 2023

@a-detiste
Copy link

I made a PR for six. I do like too provide a bit of help to random projects:

https://wiki.debian.org/Python3-six-removal

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

No branches or pull requests

3 participants