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 python version requirements to setup.py #1093
Conversation
This is need in the future so we can gracefully remove python 2.7 support. See #1075.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We probably want to specify the 2.7 requirement, if that is an important step in pypi giving out 1.x as the last python2.7 release. The other deprecations seem separate however. Is there any reason we're removing those here?
The python3 versions that were dropped are because sphinx has already dropped support for them when sphinx 1.6 is removed. We require sphinx 1.6 because it is the first version with experimental HTML5 and needed for some other bug fixes in the theme. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm not an expert in the theme but this pull request doesn't do anything revolutionary. Python 3.0 to 3.2 are universally recognized as Bad™, and 3.3 is just too old at this point. As @Blendify has clarified, these versions haven't been supported since Sphinx 1.5 anyway. There are no outstanding discussions left. In summary: I'm merging it. Thanks @Blendify for your persistence!
This is need in the future so we can gracefully remove python 2.7 support.
See #1075.
The required python versions are based on sphinx 1.6 which is our minimal required version.
https://packaging.python.org/guides/dropping-older-python-versions/