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

Use pyproject.toml #32

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Use pyproject.toml #32

wants to merge 1 commit into from

Conversation

edsu
Copy link

@edsu edsu commented Jul 29, 2024

setuptools has deprecated the test command for the past five years and recently removed it. This caused quite a stir since many people's builds broke.

Our build of pywb failed because surt is still using setuptools.command.test. After the confusion setuptools decided to yank this version from pypi while they determine whether there is a less painful way to remove the test command.

Perhaps this is a good time to remove the dependency on setuptools.commands.test, and given adoption of PEP-621 also switch over to using a pyproject.toml file?

fixes #33

setuptools has [deprecated] the test command for the past five years and
recently removed it. This caused [quite a stir] since many people's
build broke.

Our build of [pywb] failed because surt is still using setuptools.test.
I thought perhaps this would be a good time to removing that dependency
and given adoption of [PEP-621] perhaps now is a good time to switch
over to using a `pyproject.toml` file?

[deprecated]: pypa/setuptools#1684
[quite a stir]: pypa/setuptools#4519
[pywb]: https://github.com/webrecorder/pywb
[PEP-621]: https://peps.python.org/pep-0621/
@@ -6,7 +6,7 @@
[tox]
envlist =
py26, py27,
py33, py34, py35, py36
py33, py34, py35, py36, py312
Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I added this to verify that I can run the tests with tox.

@edsu
Copy link
Author

edsu commented Jul 30, 2024

Alternatively perhaps #34 is preferable if you'd rather not use pyproject.toml

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.

Remove setuptools.command.test
1 participant