[Twisted-Python] Twisted 21.7.0 Pre-Release Announcement
Glyph
glyph at twistedmatrix.com
Fri Jul 16 13:27:01 MDT 2021
> On Jul 16, 2021, at 2:20 AM, Richard van der Hoff <richard at matrix.org> wrote:
>
> On 16/07/2021 00:18, Glyph wrote:
>
>>
>>
>>> On Jul 15, 2021, at 9:00 AM, Richard van der Hoff <richard at matrix.org <mailto:richard at matrix.org>> wrote:
>>>
>>> We can't just go and add type annotations because we need to maintain compatibility with older Twisted (to make it possible to package in Debian et al).
>>>
>>> Any suggestions for keeping mypy happy?
>>>
>>
>> Are you saying you need it to typecheck against older versions or just run against them?
> Ah, this gave me the clue I needed. We just need to run against them. Which means I can put type hints in comments, where they will be ignored at runtime. It's fiddly, but it will work well enough.
>
You can also do 'from __future__ import annotations' to avoid the annotations getting evaluated, which might be slightly less awkward.
> Thanks Glyph, and thanks to Adi and Barry for your suggestions too.
>
> One particular problem I came across was the type annotation on inlineCallbacks. I've filed https://twistedmatrix.com/trac/ticket/10231 <https://twistedmatrix.com/trac/ticket/10231> about it - would appreciate thoughts.
>
This definitely looks wrong; there should be a TypeVar in there. Adi, I'd go so far as to say that this should be a release blocker, although the change should be fairly minimal.
Richard, could you please make a proper PR for this to get CI kicked off and make sure the new annotation doesn't cause any failures?
-g
-------------- next part --------------
An HTML attachment was scrubbed...
URL: </pipermail/twisted-python/attachments/20210716/1ceb81a5/attachment.htm>
More information about the Twisted-Python
mailing list