Skip to content

Specify class returning itself in Logger.__get__.#1541

Merged
wsanchez merged 2 commits intotwisted:trunkfrom
jchristgit:10119-jchristgit-twisted-logger-get-return-type
Mar 8, 2021
Merged

Specify class returning itself in Logger.__get__.#1541
wsanchez merged 2 commits intotwisted:trunkfrom
jchristgit:10119-jchristgit-twisted-logger-get-return-type

Conversation

@jchristgit
Copy link
Contributor

@jchristgit jchristgit commented Mar 8, 2021

Scope and purpose

This pull request aims to fix a type error reported by mypy when checking classes with a class attribute instantiating twisted.logger.Logger.

Contributor Checklist:

  • The associated ticket in Trac is here: https://twistedmatrix.com/trac/ticket/10119
  • I ran tox -e lint to format my patch to meet the Twisted Coding Standard
  • I have created a newsfragment in src/twisted/newsfragments/ (see: News files)
  • I have updated the automated tests. (I am not sure how to add any, as this is a mypy change. Open to suggestions.)
  • I have submitted the associated Trac ticket for review by adding the word review to the keywords field in Trac, and putting a link to this PR in the comment; it shows up in https://twisted.reviews/ now.

@jchristgit jchristgit force-pushed the 10119-jchristgit-twisted-logger-get-return-type branch from 5882fe4 to 070c0e5 Compare March 8, 2021 16:16
@graingert graingert requested a review from wsanchez March 8, 2021 16:59
@wsanchez wsanchez merged commit f93f647 into twisted:trunk Mar 8, 2021
@jchristgit jchristgit deleted the 10119-jchristgit-twisted-logger-get-return-type branch March 8, 2021 22:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants