Skip to content

Missing type annotations for properties on logging.LoggerAdapter #5015

Closed

Description

@yoavp10

Hi,
LoggerAdapter class have 2 new properties starting from version 3.6*:
manager (getter and setter)
name (getter)

This have no equivalent in typeshed, I'd like to create them. What do you think?

*Unfortunately these properties can't be found on the official cpython docs, however I found the relevant python bug and pr's:
https://bugs.python.org/issue31457
python/cpython@cf2a4f9
python/cpython@fee0475
I opened a docs bug for cpython and plan on making a PR on this front too.

Activity

  1. Akuli commented on Feb 10, 2021

    @Akuli
    Collaborator

    I believe the typeshed convention is to add attributes like this with a comment # undocumented. Even pull requests adding _non_public attributes are accepted when someone needs those non-public attributes.

  2. changed the title [-]Missing type annotations for properties on logging.LoggerAdapter[/-] [+]Missing type annotations for properties on logging.LoggerAdapter #undocumented[/+] on Feb 10, 2021
  3. changed the title [-]Missing type annotations for properties on logging.LoggerAdapter #undocumented[/-] [+]Missing type annotations for properties on logging.LoggerAdapter # undocumented[/+] on Feb 10, 2021
  4. changed the title [-]Missing type annotations for properties on logging.LoggerAdapter # undocumented[/-] [+]Missing type annotations for properties on logging.LoggerAdapter[/+] on Feb 10, 2021
  5. Akuli commented on May 12, 2021

    @Akuli
    Collaborator

    fixed in #5388

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions