Closed sevein closed 4 years ago
@carlwilson, I'm not authorized to merge. Can you do it? I had to push a linter fix.
@carlwilson, I'm not authorized to merge. Can you do it? I had to push a linter fix.
We can have a look at setting up a proper review and test cycle on here soon I think. Once passed tests and a review is compulsory we can open up the merge list a little.
fido does not need setuptools to run correctly and the requirement is considered unsafe.
I don't expect this to be a breaking change but I'd be curious to hear your thoughts. I've found that pytest was also listing setuptools but they stopped doing so in pytest v4.6.0 (commit https://github.com/pytest-dev/pytest/commit/13f02af97d676bdf7143aa1834c8898fbf753d87), presumably because they were already using
setup_requires
.Fixes #187.