search

MatthiasValvekens / certomancer

Quickly construct, mock & deploy PKI test configurations using simple declarative configuration.
MIT License
26 stars 4 forks source link

It requires `pytest-runner` for build #10

Closed catap closed 9 months ago

catap commented 9 months ago

Let me quote pyproject.toml:

[build-system]
requires = [
    "setuptools>=67.4",
    "wheel", "pytest-runner",
]

is it really required to build it, not test?

MatthiasValvekens commented 9 months ago

This used to be a requirement for being able to use setup.py test. But direct setup.py invocation is deprecated anyway, so it can probably be removed. I think this is mostly a leftover from the automatic migration to pyproject.toml.

catap commented 9 months ago

Thanks for very fast fix!

catap commented 9 months ago

The same issue exists at https://github.com/MatthiasValvekens/certvalidator and https://github.com/MatthiasValvekens/pyHanko at least