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 nose-related tests #171

Open
mcepl opened this issue Dec 11, 2020 · 5 comments
Open

Remove nose-related tests #171

mcepl opened this issue Dec 11, 2020 · 5 comments

Comments

@mcepl
Copy link
Contributor

mcepl commented Dec 11, 2020

I am a maintainer of Python packages in openSUSE, and I am on my crusade of eliminating nose1 from our distribution. When I look at its repository on https://github.com/nose-devs/nose, the last release 1.3.7 was on 2 Jun 2015, and even the last commit on the master branch was on 4 Mar 2016. Moreover it will stop working with Python 3.10.

This patch eliminates dependency on nose, which has been unnecessary for a long time.

@stratakis
Copy link
Contributor

That would be great. Indeed nose is dead for quite some time, maybe changing everything to pytest would be sufficient.

@mcepl
Copy link
Contributor Author

mcepl commented Jan 22, 2021

Actually, you don’t have to even do that. My patch is just plain unittest.

@stratakis
Copy link
Contributor

@Jeff-Meadows pinging here for visibility.

Similar to mcepl I'm one of the Python maintainers for Fedora. We will soon face the same issue on Fedora when we update to Python 3.10 (for which we already test the whole distro against) and removing nose would be of great benefit to us as well.

We have also deprecated nose since the previous Fedora release: https://fedoraproject.org/wiki/Changes/DeprecateNose

@Jeff-Meadows
Copy link
Contributor

Thanks for the ping. I agree there seems to be no point in continuing to support nose.

I'd be happy to review and merge a PR that accomplished the task.

@mcepl
Copy link
Contributor Author

mcepl commented Jan 22, 2021

I'd be happy to review and merge a PR that accomplished the task.

You have patch attached to the first comment of this issue. Sorry, I work with 2000+ Python packages (couple of hundred of them were dependent on nose), so I don't have enough time to clone, fork, and create pull requests for each of them.

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