<div dir="ltr">Hi,<div><br></div><div>I reviewed your branch: <span style="font-size:12.8px"> </span><a href="https://github.com/wrohdewald/twisted/commits/spread-py3-7598/twisted" rel="noreferrer" target="_blank" style="font-size:12.8px">https://github.com/wrohdewald/twisted/commits/spread-py3-7598/twisted</a></div><div><br></div><div>In my opinion, that branch has two categories of Python 3 porting:</div><div><br></div><div>Category 1: Minor</div><div>  -> these are "minor" fixos, such as dealing with xrange -> range</div><div><br></div><div>Category 2: Major</div><div>   -> you have done a lot of work to deal with bytes versus unicode strings</div><div><br></div><div>I hope you haven't lost interest in this, and are still willing to push forward.</div><div>I would recommend that if you can separate out the minor fixes</div><div>into separate GitHub pull requests and separate Trac tickets, then do that.</div><div><br></div><div>If you see: <a href="https://github.com/twisted/twisted/commits/trunk">https://github.com/twisted/twisted/commits/trunk</a></div><div><br></div><div>you will see I have submitted a lot of pull requests + trac tickets of "minor"</div><div>Python 3 fixes, and those have successfully been reviewed and merged</div><div>by Twisted committers.</div><div><br></div><div>Dealing with bytes versus unicode strings will take longer to review,</div><div>because it is more advanced.  That is why I recommend separating</div><div>out the "minor" fixos if you can.</div><div><br></div><div>The other think I would recommend is if you do periodic merges</div><div>from trunk to your branch, since a lot of new Python 3 fixes have gone into trunk in the past few weeks.</div><div><br></div><div>--</div><div>Craig</div></div>