Skip to content
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

bpo-37759: More updates to Whatsnew 3.8 #16854

Merged
merged 17 commits into from Oct 20, 2019

Conversation

rhettinger
Copy link
Contributor

@rhettinger rhettinger commented Oct 19, 2019

rhettinger added 2 commits Oct 20, 2019
Other minor fixes as well.
Also, dedup the __reduce__ entry.
Doc/whatsnew/3.8.rst Show resolved Hide resolved
Doc/whatsnew/3.8.rst Show resolved Hide resolved
Doc/whatsnew/3.8.rst Show resolved Hide resolved
Doc/whatsnew/3.8.rst Show resolved Hide resolved
@rhettinger rhettinger merged commit c93883c into python:master Oct 20, 2019
@rhettinger rhettinger deleted the whatsnew_3_8_updates branch Oct 20, 2019
@miss-islington
Copy link
Contributor

miss-islington commented Oct 20, 2019

Thanks @rhettinger for the PR 🌮🎉.. I'm working now to backport this PR to: 3.8.
🐍🍒🤖

@bedevere-bot
Copy link

bedevere-bot commented Oct 20, 2019

GH-16867 is a backport of this pull request to the 3.8 branch.

rhettinger added a commit that referenced this pull request Oct 20, 2019
* math.perm() and math.comb()

* math.isqrt()

* Add singledispatchmethod()

* itertools.accumulate()

* Optional headers for xmlrpc.client.ServerProxy

* IDLE non-BMP characters

* import collections.abc directly

* @coroutine is deprecated

* pprint.pp()

* New options for object.__reduce__()

* DictReader no longer returns OrderedDicts

* "force" option for logging.basicConfig()

* Fix spelling

* cProfile context manager

* Various markup/grammar fixes from Kyle Stanley.
Other minor fixes as well.
Also, dedup the __reduce__ entry.

* Fix markup

* Fix grammar nits found by MS Word
(cherry picked from commit c93883c)

Co-authored-by: Raymond Hettinger <rhettinger@users.noreply.github.com>
Copy link
Member

@serhiy-storchaka serhiy-storchaka left a comment

Great work! Thank you, Raymond!

But I think there are some issues with using the tilde.

element is a callable with a ``(obj, state)`` signature. This allows the
direct control over the state-updating behavior of a specific object. If
not *None*, this callable will have priority over the object's
:meth:`~__setstate__` method.
Copy link
Member

@serhiy-storchaka serhiy-storchaka Oct 20, 2019

Choose a reason for hiding this comment

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

~ is not needed here. It could be :meth:`~object.__setstate__`, but maybe :meth:`__setstate__` is enough.

Copy link
Member

@aeros aeros Oct 20, 2019

Choose a reason for hiding this comment

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

:meth:`__setstate__`

IIRC, the above does not correctly generate an inline link. This applies to the other dunder methods and attributes inherited from object as well, it has to be:

:meth:`object.__setstate__`

Usually, I've used:

:meth:`~object.__setstate__`

since only the __setstate__ part is relevant for the context of the readers.

-------

Added a *force* keyword argument to :func:`logging.basicConfig()`
When set to *True*, any existing handlers attached
Copy link
Member

@serhiy-storchaka serhiy-storchaka Oct 20, 2019

Choose a reason for hiding this comment

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

It is usually written as ``True``.

@@ -973,8 +1109,8 @@ The :func:`socket.if_nameindex()`, :func:`socket.if_nametoindex()`, and
ssl
---

Added :attr:`ssl.SSLContext.post_handshake_auth` to enable and
:meth:`ssl.SSLSocket.verify_client_post_handshake` to initiate TLS 1.3
Added :attr:`~ssl.SSLContext.post_handshake_auth` to enable and
Copy link
Member

@serhiy-storchaka serhiy-storchaka Oct 20, 2019

Choose a reason for hiding this comment

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

The module is known from the context, but it is not clear what classes the attribute and the method belong to. Maybe you meant :attr:`SSLContext.post_handshake_auth <ssl.SSLContext.post_handshake_auth>`?

There are similar issues with some other uses of the tilde. It lefts only the last component.

jacobneiltaylor pushed a commit to jacobneiltaylor/cpython that referenced this pull request Dec 5, 2019
* math.perm() and math.comb()

* math.isqrt()

* Add singledispatchmethod()

* itertools.accumulate()

* Optional headers for xmlrpc.client.ServerProxy

* IDLE non-BMP characters

* import collections.abc directly

* @coroutine is deprecated

* pprint.pp()

* New options for object.__reduce__()

* DictReader no longer returns OrderedDicts

* "force" option for logging.basicConfig()

* Fix spelling

* cProfile context manager

* Various markup/grammar fixes from Kyle Stanley.
Other minor fixes as well.
Also, dedup the __reduce__ entry.

* Fix markup

* Fix grammar nits found by MS Word
shihai1991 pushed a commit to shihai1991/cpython that referenced this pull request Jan 31, 2020
* math.perm() and math.comb()

* math.isqrt()

* Add singledispatchmethod()

* itertools.accumulate()

* Optional headers for xmlrpc.client.ServerProxy

* IDLE non-BMP characters

* import collections.abc directly

* @coroutine is deprecated

* pprint.pp()

* New options for object.__reduce__()

* DictReader no longer returns OrderedDicts

* "force" option for logging.basicConfig()

* Fix spelling

* cProfile context manager

* Various markup/grammar fixes from Kyle Stanley.
Other minor fixes as well.
Also, dedup the __reduce__ entry.

* Fix markup

* Fix grammar nits found by MS Word
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.

None yet

6 participants