]> git.ipfire.org Git - thirdparty/Python/cpython.git/commitdiff
Enhance TypedDict docs around required/optional keys (#109547)
authorJelle Zijlstra <jelle.zijlstra@gmail.com>
Wed, 27 Sep 2023 17:35:46 +0000 (10:35 -0700)
committerGitHub <noreply@github.com>
Wed, 27 Sep 2023 17:35:46 +0000 (10:35 -0700)
As discussed in comments to #109544, the semantics of this attribute
are somewhat confusing. Add a note explaining its limitations and
steering users towards __required_keys__ and __optional_keys__ instead.

Doc/library/typing.rst

index e63b839931822c8b3872144a0804aaec3bea759d..8f6919322012257845da486a166c89eb6da7e767 100644 (file)
@@ -2404,6 +2404,13 @@ types.
          >>> Point3D.__total__
          True
 
+      This attribute reflects *only* the value of the ``total`` argument
+      to the current ``TypedDict`` class, not whether the class is semantically
+      total. For example, a ``TypedDict`` with ``__total__`` set to True may
+      have keys marked with :data:`NotRequired`, or it may inherit from another
+      ``TypedDict`` with ``total=False``. Therefore, it is generally better to use
+      :attr:`__required_keys__` and :attr:`__optional_keys__` for introspection.
+
    .. attribute:: __required_keys__
 
       .. versionadded:: 3.9
@@ -2439,6 +2446,14 @@ types.
 
       .. versionadded:: 3.9
 
+      .. note::
+
+         If ``from __future__ import annotations`` is used or if annotations
+         are given as strings, annotations are not evaluated when the
+         ``TypedDict`` is defined. Therefore, the runtime introspection that
+         ``__required_keys__`` and ``__optional_keys__`` rely on may not work
+         properly, and the values of the attributes may be incorrect.
+
    See :pep:`589` for more examples and detailed rules of using ``TypedDict``.
 
    .. versionadded:: 3.8