Skip to content

bpo-34160: Update news entry for XML order attributes#12335

Merged
rhettinger merged 3 commits intopython:masterfrom
dfrojas:newsentry-xmlattributeorder
Mar 16, 2019
Merged

bpo-34160: Update news entry for XML order attributes#12335
rhettinger merged 3 commits intopython:masterfrom
dfrojas:newsentry-xmlattributeorder

Conversation

@dfrojas
Copy link
Copy Markdown
Contributor

@dfrojas dfrojas commented Mar 14, 2019

Update news entry for change of Python API which preserves the XML order attributes.

https://bugs.python.org/issue34160

@dfrojas dfrojas changed the title Update news entry for XML order attributes bpo-34160: Update news entry for XML order attributes Mar 14, 2019
@dfrojas dfrojas force-pushed the newsentry-xmlattributeorder branch from e75bce7 to 03f8ddb Compare March 15, 2019 03:55
Comment thread Doc/whatsnew/3.8.rst Outdated
changing the selection. (Contributed by Serhiy Storchaka in :issue:`31508`.)

* The :meth:`writexml`, :meth:`toxml` and :meth:`toprettyxml` methods
now preserves the attribute order specified by the user.
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.

please list which modules have been modified. Something like: "xml.dom.minidom, xml.etree, ...: The ... methods ...".

Comment thread Doc/whatsnew/3.8.rst Outdated
changing the selection. (Contributed by Serhiy Storchaka in :issue:`31508`.)

* The :meth:`writexml`, :meth:`toxml` and :meth:`toprettyxml` methods
now preserves the attribute order specified by the user.
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.

Plural verb form: “The . . . methods now preserve the . . .”

@rhettinger rhettinger merged commit 06e1e68 into python:master Mar 16, 2019
@bedevere-bot
Copy link
Copy Markdown

@rhettinger: Please replace # with GH- in the commit message next time. Thanks!

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.

7 participants