Opened 3 years ago

Closed 3 years ago

#6060 enhancement closed fixed (fixed)

Port twisted.internet.protocol to Python 3

Reported by: itamar Owned by: itamar
Priority: normal Milestone: Python 3.3 Minimal
Component: core Keywords:
Cc: Branch: branches/protocol-py3-6060
(github, patch, buildbot, log)
Author: itamarst Launchpad Bug:


twisted.internet.protocol should run on Python 3.

Change History (5)

comment:1 Changed 3 years ago by itamarst

  • Author set to itamarst
  • Branch set to branches/protocol-py3-6060

(In [35952]) Branching to 'protocol-py3-6060'

comment:2 Changed 3 years ago by itamar

  • Keywords review added
  • Owner changed from itamar to exarkun

I improved test coverage somewhat, and moved over a couple of tests from twisted.test.test_protocols to a better location (they were otherwise unchanged). I opened a separate ticket (#6064) for FileWrapper test coverage. has been started.

comment:3 Changed 3 years ago by itamar

Oh, and I should note I didn't do tests for the datagram and process classes since they're (somewhat) tested by UDP and process code.

comment:4 Changed 3 years ago by exarkun

  • Keywords review removed
  • Owner changed from exarkun to itamar

Woo! Tests yay. Thanks.

  1. __future__ imports in twisted.internet.protocol
  2. AdapterTests.test_consumerToProtocol docstring says IConsumer where it means IProtocol
  3. You're also marked as the maintainer of twisted.internet.protocol, in case you wanted to do something about that. :)

That's all. Please merge when these are addressed.

comment:5 Changed 3 years ago by itamarst

  • Resolution set to fixed
  • Status changed from new to closed

(In [36029]) Merge protocol-py3-6008.

Author: itamar Review: exarkun Fixes: #6060

Port twisted.internet.protocol to Python 3.

Note: See TracTickets for help on using tickets.