-
Notifications
You must be signed in to change notification settings - Fork 3.1k
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
Only show Python 2.7 EOL date on CPython. #6208
Conversation
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.
Looks good, @pradyunsg @dstufft @pfmoore however given that most users are stuck on Python 2 because of their system limitations, not their personal choice, maybe we should drop the message entirely. Guido seems to agree towards this too.
So as I mentioned on the other issue, in case my vote counts :), assuming pip has no concrete plans to drop Py2 support, I'd personally:
with the reasoning being that whether they like it or not, users using CPython do need to consider EOL (whether pip is the right thing to warn about that who knows, which maybe is the argument to drop it entirely, but I hope something does show one...) Users using PyPy (well me) do not need to worry about EOL, there isn't one, and our support is continuing for now. If |
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.
Will include in 19.0.2.
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.
Can a .bugfix
news file be added since this is a user-visible change, and I think the distinction is worth highlighting in the release notes?
Other Python implementations, notably PyPy, are not EOL'ing on January 1, 2020.
e0f9386
to
77c989f
Compare
@cjerdonek done |
I'm inclined to add an option and suggest in the message how to suppress the message itself, using |
@pradyunsg Is it okay to discuss this in a separate issue / PR so this can be merged? |
Yea, definitely. |
This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs. |
Other Python implementations, notably PyPy, are not EOL'ing on January
1, 2020.
I've left the core of the warning intact, and have just found https://discuss.python.org/t/packaging-and-python-2/662/73 about pip's own plans (obviously my hope would be that pip does continue to support py2, but will follow that along).
Closes: #6207