-
-
Notifications
You must be signed in to change notification settings - Fork 32.1k
gh-122519: Adding socket module shutdown() constants description #122543
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
Conversation
Modify some grammar Adding availability
ace016e
to
567ef43
Compare
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.
Funny that Sphinx didn't complain about the fact that the constants where not defined previously. I'm ok with those changes but a core dev needs to approve it.
By the way, no need to merge the main branch into your branch. It's better to only merge the main branch if there are conflicts and you need to solve those conflicts (otherwise, I would be notified of such action, thinking that there was an actual change). |
Understood! Sorry I didn`t noticed that. |
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 to me
Should we merge this ? |
Thanks @Damien-Chen for the PR, and @kumaraditya303 for merging it 🌮🎉.. I'm working now to backport this PR to: 3.12, 3.13. |
pythonGH-122543) (cherry picked from commit 8a59dec) Co-authored-by: Damien <[email protected]>
GH-123093 is a backport of this pull request to the 3.13 branch. |
pythonGH-122543) (cherry picked from commit 8a59dec) Co-authored-by: Damien <[email protected]>
GH-123094 is a backport of this pull request to the 3.12 branch. |
According to ##122519, adding constant argument description for
socket.shutdown()
method.📚 Documentation preview 📚: https://cpython-previews--122543.org.readthedocs.build/