Thanks to visit codestin.com
Credit goes to github.com

Skip to content

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

Merged
merged 4 commits into from
Aug 17, 2024

Conversation

Damien-Chen
Copy link
Contributor

@Damien-Chen Damien-Chen commented Aug 1, 2024

According to ##122519, adding constant argument description for socket.shutdown() method.


📚 Documentation preview 📚: https://cpython-previews--122543.org.readthedocs.build/

@bedevere-app bedevere-app bot added docs Documentation in the Doc dir skip news awaiting review labels Aug 1, 2024
@Damien-Chen Damien-Chen changed the title gh-#122519: Adding socket module shutdown() constants description gh-122519: Adding socket module shutdown() constants description Aug 1, 2024
Modify some grammar

Adding availability
@Damien-Chen Damien-Chen force-pushed the main branch 2 times, most recently from ace016e to 567ef43 Compare August 1, 2024 11:35
@Damien-Chen Damien-Chen requested a review from picnixz August 1, 2024 13:25
Copy link
Member

@picnixz picnixz left a 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.

@picnixz
Copy link
Member

picnixz commented Aug 2, 2024

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).

@Damien-Chen
Copy link
Contributor Author

Understood! Sorry I didn`t noticed that.

@Damien-Chen Damien-Chen requested a review from vadmium August 2, 2024 08:29
@Damien-Chen Damien-Chen requested a review from vadmium August 2, 2024 09:22
Copy link
Member

@vadmium vadmium left a 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

@Damien-Chen
Copy link
Contributor Author

Should we merge this ?

@kumaraditya303 kumaraditya303 added the needs backport to 3.12 only security fixes label Aug 17, 2024
@kumaraditya303 kumaraditya303 added the needs backport to 3.13 bugs and security fixes label Aug 17, 2024
@kumaraditya303 kumaraditya303 merged commit 8a59dec into python:main Aug 17, 2024
27 checks passed
@miss-islington-app
Copy link

Thanks @Damien-Chen for the PR, and @kumaraditya303 for merging it 🌮🎉.. I'm working now to backport this PR to: 3.12, 3.13.
🐍🍒⛏🤖

miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Aug 17, 2024
@bedevere-app
Copy link

bedevere-app bot commented Aug 17, 2024

GH-123093 is a backport of this pull request to the 3.13 branch.

@bedevere-app bedevere-app bot removed the needs backport to 3.13 bugs and security fixes label Aug 17, 2024
miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Aug 17, 2024
@bedevere-app
Copy link

bedevere-app bot commented Aug 17, 2024

GH-123094 is a backport of this pull request to the 3.12 branch.

@bedevere-app bedevere-app bot removed the needs backport to 3.12 only security fixes label Aug 17, 2024
kumaraditya303 pushed a commit that referenced this pull request Aug 17, 2024
…on (GH-122543) (#123094)

gh-122519: Adding socket module shutdown() constants description (GH-122543)
(cherry picked from commit 8a59dec)

Co-authored-by: Damien <[email protected]>
kumaraditya303 pushed a commit that referenced this pull request Aug 17, 2024
…on (GH-122543) (#123093)

gh-122519: Adding socket module shutdown() constants description (GH-122543)
(cherry picked from commit 8a59dec)

Co-authored-by: Damien <[email protected]>
jeremyhylton pushed a commit to jeremyhylton/cpython that referenced this pull request Aug 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
docs Documentation in the Doc dir skip news
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants