FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

bpo-36543: Restore cElementTree and mark it for removal in 3.10 by tiran · Pull Request #19921 · python/cpython · GitHub

/ cpython Public

bpo-36543: Restore cElementTree and mark it for removal in 3.10 - #19921

Closed
tiran wants to merge 2 commits into
python:masterfrom
tiran:restore_celementtree
Closed

bpo-36543: Restore cElementTree and mark it for removal in 3.10#19921
tiran wants to merge 2 commits into
python:masterfrom
tiran:restore_celementtree

Conversation

tiran commented May 5, 2020
edited
Loading

Copy link
Copy Markdown
Member
  • Restore xml.etree.cElementTree
  • Schedule the module for removal in 3.10
  • Add a PendingDeprecationWarning

https://bugs.python.org/issue36543

tiran changed the title bpo-36453: Restore cElementTree and mark it for removal 3.10 bpo-36453: Restore cElementTree and mark it for removal in 3.10 May 5, 2020
tiran force-pushed the restore_celementtree branch from c10d287 to 1372fe1 Compare May 5, 2020 11:57
tiran changed the title bpo-36453: Restore cElementTree and mark it for removal in 3.10 bpo-36543: Restore cElementTree and mark it for removal in 3.10 May 5, 2020

scoder left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

LGTM.
Do we need a News entry for this? It's only reverting an alpha change.

Comment thread Doc/whatsnew/3.9.rst Outdated

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

A typo in your own name? That's rare, I guess. ;-)

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

You wouldn't believe me 👍

Comment thread Lib/xml/etree/cElementTree.py Outdated

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality
Suggested change
)
)
del _warn

scoder commented May 5, 2020

Copy link
Copy Markdown
Contributor

Let's got with GH-19920 instead, shall we?

tiran force-pushed the restore_celementtree branch from 1372fe1 to 23bb770 Compare May 15, 2020 17:38
Signed-off-by: Christian Heimes <christian@python.org>
tiran force-pushed the restore_celementtree branch from 23bb770 to 99cd4ef Compare May 15, 2020 17:41

Copy link
Copy Markdown
Member

With 3.9 branch created the PR pointed to master will go in for 3.10.

tiran added a commit to tiran/pip that referenced this pull request May 20, 2020
It's been deprecated and will be removed in 3.9 or 3.10. 3.9.0b1 doesn't
have cElementTree. I'd like to bring it back with a deprecation warning
to drop in 3.10.

See: python/cpython#19921
Signed-off-by: Christian Heimes <christian@python.org>
Comment thread Doc/whatsnew/3.9.rst
(Contributed by Carl Meyer in :issue:`40360`.)

* The :mod:`xml.etree.cElementTree` module has been deprecated since Python
3.3 and is scheduled to be removed in Python 3.10.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

Use more direct wording:

  • clearer
  • similar to that in xml.etree.elementtree.rst
  • fewer words = more readable
Suggested change
3.3 and is scheduled to be removed in Python 3.10.
3.3 and will be removed in Python 3.10.

gsnedders pushed a commit to gsnedders/html5lib-python that referenced this pull request Jun 10, 2020
It's been deprecated and will be removed in 3.9 or 3.10. 3.9.0b1 doesn't
have cElementTree. I'd like to bring it back with a deprecation warning
to drop in 3.10.

See: python/cpython#19921
Signed-off-by: Christian Heimes <christian@python.org>
Signed-off-by: Sam Sneddon <me@gsnedders.com>

scoder commented Jun 10, 2020

Copy link
Copy Markdown
Contributor

I merged #20117, which reverts the removal.

scoder closed this Jun 10, 2020
gsnedders pushed a commit to gsnedders/html5lib-python that referenced this pull request Jun 17, 2020
It's been deprecated and will be removed in 3.9 or 3.10. 3.9.0b1 doesn't
have cElementTree. I'd like to bring it back with a deprecation warning
to drop in 3.10.

See: python/cpython#19921
Signed-off-by: Christian Heimes <christian@python.org>
Signed-off-by: Sam Sneddon <me@gsnedders.com>
gsnedders pushed a commit to html5lib/html5lib-python that referenced this pull request Jun 17, 2020
It's been deprecated and will be removed in 3.9 or 3.10. 3.9.0b1 doesn't
have cElementTree. I'd like to bring it back with a deprecation warning
to drop in 3.10.

See: python/cpython#19921
Signed-off-by: Christian Heimes <christian@python.org>
Signed-off-by: Sam Sneddon <me@gsnedders.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants


Back | FazBrowse Home | New Git URL