Skip to content

bpo-47022: Document asynchat, asyncore and smtpd removals in 3.12 - #31891

Merged
miss-islington merged 3 commits into
python:mainfrom
hugovk:pep594-document-3.12-removals
Mar 18, 2022
Merged

bpo-47022: Document asynchat, asyncore and smtpd removals in 3.12#31891
miss-islington merged 3 commits into
python:mainfrom
hugovk:pep594-document-3.12-removals

Conversation

@hugovk

@hugovkhugovk commented Mar 15, 2022•
edited by miss-islington
Loading

Copy link
Copy Markdown
Member

Document the deprecation of asyncore, asynchat, and smtpd with a slated removal in Python 3.12 thanks to PEP 594.

Automerge-Triggered-By: GH:brettcannon

@hugovk
hugovk requested a review from a team as a code ownerMarch 15, 2022 07:50
@hugovk

This comment was marked as resolved.

Sign in to view
@hugovkhugovk added the docs Documentation in the Doc dir label Mar 15, 2022
@hugovk
hugovkforce-pushed the pep594-document-3.12-removals branch from 662a963 to 3eb9ba8CompareMarch 15, 2022 10:05
@hugovk
hugovk requested review from brettcannon and tiranMarch 15, 2022 10:49
@brettcannon

brettcannon commented Mar 15, 2022

Copy link
Copy Markdown
Member

Two things.

One, can you add a directive to the docs like:

cpython/Doc/library/imp.rst

Line 6 in 099f756

:deprecated:

? That's what makes a module say "deprecated" in the module index.

Two, the SC wanted to leave out target removal versions in the docs in case we have to change something due to feedback (in the code itself via the warnings.warn call still makes sense). But since these modules were already deprecated and the community has not come back asking to save them I think specifying 3.12 is safe.

@hugovk

hugovk commented Mar 16, 2022

Copy link
Copy Markdown
MemberAuthor

Two things.

One, can you add a directive to the docs like:

cpython/Doc/library/imp.rst

Line 6 in 099f756

:deprecated:

? That's what makes a module say "deprecated" in the module index.

Done!

Also added to the Superseded Modules page listing deprecated modules (imp is there), putting those needing most attention first (to be removed soonest, and most recently added).

Two, the SC wanted to leave out target removal versions in the docs in case we have to change something due to feedback (in the code itself via the warnings.warn call still makes sense). But since these modules were already deprecated and the community has not come back asking to save them I think specifying 3.12 is safe.

👍

@miss-islington
miss-islington merged commit 7747384 into python:mainMar 18, 2022
@miss-islington

miss-islington commented Mar 18, 2022

Copy link
Copy Markdown
Contributor

Thanks @hugovk for the PR 🌮🎉.. I'm working now to backport this PR to: 3.9, 3.10.
🐍🍒⛏🤖

@miss-islington

miss-islington commented Mar 18, 2022

Copy link
Copy Markdown
Contributor

Sorry, @hugovk, I could not cleanly backport this to 3.10 due to a conflict.
Please backport using cherry_picker on command line.
cherry_picker 77473846439b8a3eae66de1a1cfe931619f38513 3.10

@miss-islingtonmiss-islington self-assigned this Mar 18, 2022
@miss-islington

miss-islington commented Mar 18, 2022

Copy link
Copy Markdown
Contributor

Sorry @hugovk, I had trouble checking out the 3.9 backport branch.
Please backport using cherry_picker on command line.
cherry_picker 77473846439b8a3eae66de1a1cfe931619f38513 3.9

@hugovk
hugovk deleted the pep594-document-3.12-removals branch March 19, 2022 20:39
hugovk added a commit to hugovk/cpython that referenced this pull request Mar 19, 2022
@hugovk
[3.10] bpo-47022: Document asynchat, asyncore and smtpd removals in 3…
f5751f7
….12 (pythonGH-31891)
Document the deprecation of asyncore, asynchat, and smtpd with a slated removal in Python 3.12 thanks to PEP 594..
(cherry picked from commit 7747384)
Co-authored-by: Hugo van Kemenade <hugovk@users.noreply.github.com>
@hugovkhugovk mentioned this pull request Mar 19, 2022
Merged
@bedevere-bot

bedevere-bot commented Mar 19, 2022

Copy link
Copy Markdown

GH-31997 is a backport of this pull request to the 3.10 branch.

@bedevere-botbedevere-bot removed the needs backport to 3.10 only security fixes label Mar 19, 2022
@bedevere-bot

bedevere-bot commented Mar 19, 2022

Copy link
Copy Markdown

GH-31998 is a backport of this pull request to the 3.9 branch.

asvetlov pushed a commit that referenced this pull request Mar 20, 2022
@hugovk
[3.10]bpo-47022: Document asynchat, asyncore and smtpd removals in 3…
94f038c
….12 (GH-31891) (#31997)
Document the deprecation of asyncore, asynchat, and smtpd with a slated removal in Python 3.12 thanks to PEP 594..
(cherry picked from commit 7747384)
Co-authored-by: Hugo van Kemenade <hugovk@users.noreply.github.com>
hugovk added a commit to hugovk/cpython that referenced this pull request Mar 22, 2022
@hugovk
[3.9] bpo-47022: Document asynchat, asyncore and smtpd removals in 3.…
5419793
…12 (pythonGH-31891)
Document the deprecation of asyncore, asynchat, and smtpd with a slated removal in Python 3.12 thanks to PEP 594..
(cherry picked from commit 7747384)
Co-authored-by: Hugo van Kemenade <hugovk@users.noreply.github.com>
brettcannon pushed a commit that referenced this pull request Mar 22, 2022
@hugovk
[3.9]bpo-47022: Document asynchat, asyncore and smtpd removals in 3.…
af341eb
…12 (GH-31891) (#31998)
Document the deprecation of asyncore, asynchat, and smtpd with a slated removal in Python 3.12 thanks to PEP 594..
(cherry picked from commit 7747384)
Co-authored-by: Hugo van Kemenade <hugovk@users.noreply.github.com>
This was referenced Apr 11, 2022
Closed
Closed
@hugovkhugovk mentioned this pull request Apr 26, 2022
Merged
This was referenced May 9, 2022
Open
Merged
hello-adam pushed a commit to hello-adam/cpython that referenced this pull request Jun 2, 2022
@hugovk@hello-adam
[3.9] bpo-47022: Document asynchat, asyncore and smtpd removals in 3.…
bf720a3
…12 (pythonGH-31891) (python#31998)
Document the deprecation of asyncore, asynchat, and smtpd with a slated removal in Python 3.12 thanks to PEP 594..
(cherry picked from commit 7747384)
Co-authored-by: Hugo van Kemenade <hugovk@users.noreply.github.com>
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@brettcannonbrettcannonbrettcannon approved these changes
@asvetlovasvetlovasvetlov approved these changes
@tirantiranAwaiting requested review from tiran

Assignees

@miss-islingtonmiss-islington

Labels

docsDocumentation in the Doc dir

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

6 participants

@hugovk@brettcannon@miss-islington@bedevere-bot@asvetlov@the-knights-who-say-ni