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
Drop support for Python 3.4; add support for 3.8. #23
Conversation
Please don't drop support for Python 2 just yet. |
@jamadden Is this package really used outside Zope 4? |
Yes this package is used outside Zope 4. ExtensionClass is used by Acquisition, and Acquisition is usedful and used in other contexts, including generic Zope 3 applications. |
@jamadden What's your suggested schedule to remove Python 2 support in Zopefoundation packages? |
Basically, when it becomes otherwise completely untenable due to overall ecosystem changes. That's probably anywhere from six months to a year or two? It might depend on the project. Most of these packages are core foundation packages that aren't going to see, at this point, a tremendous benefit from dropping Python 2 (especially as they'll continue to support Python 3.5). So I feel that they shouldn't be the forcing function; rather, they should continue to tick along quietly as they have been so that as people eventually find their own other forcing functions, they can make the switch in their own time, knowing that the core foundation packages that they've been using are both current and compatible and won't surprise them. That is, if you've been running Let the "leaf" dependencies, the single-purpose libraries, the things that aren't deeply integrated into the core architecture of systems go first. Most zopefoundation projects should be the last things to drop support. |
@jamadden I got your point. So let's postpone dropping Python 2 support for non-leaf libraries a some months. I updated this PR accordingly. |
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.
LGTM!
No description provided.