Opened 9 years ago

Closed 9 years ago

#6236 task closed fixed (fixed)

Merge `twisted/internet/` into `twisted/internet/`

Reported by: Jean-Paul Calderone Owned by: Jean-Paul Calderone
Priority: normal Milestone:
Component: core Keywords:
Cc: Branch: branches/endpoints-merge-6236
branch-diff, diff-cov, branch-cov, buildbot
Author: exarkun


Also merge the associated test module. See #6183.

Change History (4)

comment:1 Changed 9 years ago by Jean-Paul Calderone

Author: exarkun
Branch: branches/endpoints-merge-6236

(In [36683]) Branching to 'endpoints-merge-6236'

comment:2 Changed 9 years ago by Jean-Paul Calderone

Keywords: review added
Owner: Jean-Paul Calderone deleted

Build results

Once this lands, reviewing #4696 will hopefully be easier.

comment:3 Changed 9 years ago by Tom Prince

Keywords: review removed
Owner: set to Jean-Paul Calderone

This seems to lack __future__ imports. (The test module lacked it to begin with, but the python3 guide seems to suggest it be there anyway).

There are also a couple of new blank line errors reported by twistedchecker in test_endpoint. All of the reported errors might as well be cleaned up now.

All of the changed code looks correct, and I did a quick scan of the copied code looking at shape, and things seem to be in order.

Please commit after fixing.

comment:4 Changed 9 years ago by Jean-Paul Calderone

Resolution: fixed
Status: newclosed

(In [36703]) Merge endpoints-merge-6236

Author: exarkun Reviewer: tom.prince Fixes: #6236

Re-integrate the Python 3 ported endpoints implementation into twisted/internet/

Note: See TracTickets for help on using tickets.