Hi all,
rgd Twisted v21.2.0, thanks a lot to all involved in this release! the
release, mypy, more py3, .. +1! really appreciated.
now, fwiw, just wanted to let you know (sorry, too late) about a
regression with
https://github.com/crossbario/crossbar
that actually originates from
https://github.com/crossbario/autobahn-python/issues/1470
=>
with Twisted v20.3.0 calling
twisted.internet.protocol.ProcessProtocol.getPeer raises an AttributeError
with Twisted v21.2.0 calling the same method raises an
NotImplementedError
=>
this fix broke our workaround for the bug now fixed;)
https://github.com/crossbario/autobahn-python/pull/1469/files
and released in:
https://pypi.org/project/autobahn/21.3.1/
https://pypi.org/project/crossbar/21.3.1/
Cheers,
/Tobias
It seems like this change does not actually violate Twisted's compatibility policy. This is not to say introducing such regressions is desirable, but when constructing such work-arounds it might be worth paying closer-than-average attention to what the policy actually is.
Policy aside, this change doesn't seem like much of an improvement to me. If I were to guess, I would guess the change was made to satisfy some check Mypy is now being asked to make about Twisted. If that's the case, it seems unfortunate that real-world software is suffering so that a synthetic goal can be achieved. I do recognize there is a perception that practical value can come from attending to the errors Mypy reports. It would probably benefit everyone if more care were taken to consider the real-world consequences of changes that are made to satisfy the non-real-world goalposts set by tools like Mypy.
Jean-Paul
_______________________________________________
Twisted-Python mailing list
Twisted-Python@twistedmatrix.com
https://twistedmatrix.com/cgi-bin/mailman/listinfo/twisted-python