-
Notifications
You must be signed in to change notification settings - Fork 328
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
Add minimum required Python version #200
Conversation
Add test for every alive Python version
what causes this change? for both. |
What is the oldest python version we can support? I think it is driven by the requirements, not this package. |
The oldest version is python 3.8. Everything else is EOL: |
|
I am closing this because it is not properly justified. Firstly, just because the Python project does not provide support for older versions, does not mean imagehash is incompatible. Second, passing tests is a requirement for compatible versions, but there may be compatible versions beyond the tests currently set up. We also do not test all patch versions of python, yet assume they are compatible. Imagehash is pure python code. The compatibility is introduced by dependencies. We do not need to document these dependencies here, because pip will look them up as needed. There is a cost associated to testing "all" or a large number of python versions, namely a carbon footprint on the computing center for every git push. It seems to me we do not gain substantial information here by testing intermediate versions over testing the earliest and latest supported python version. |
The statement contradicts itself, because the project does not make any statement about the actively supported Python versions.
This is simply wrong. Python code is processed by the Python interpreter and thus has a clear dependency on it.
|
Change