[Twisted-Python] a review to speed up other reviews?

Glyph Lefkowitz glyph at twistedmatrix.com
Wed Oct 8 12:54:03 MDT 2014


On Oct 8, 2014, at 11:50 AM, Jonathan Lange <jml at mumak.net> wrote:

> Comments are on the ticket: https://twistedmatrix.com/trac/ticket/7633#comment:14. I forgot to reassign when I did the review earlier today.

Thanks again.

If that was intended to be a full review though, you did leave out one important step:

As documented in <https://twistedmatrix.com/trac/wiki/ReviewProcess#Howtobeagoodreviewer>,

When you're done with the review, always say what the next step should be: for example, if the author is a committer, can they commit after making a few minor fixes? If your review feedback is more substantial, should they re-submit for another review?

It looks like a "address these minor issues and merge" review, but given that you said you didn't have time to look at certain things, I don't want to make any assumptions.

-g
-------------- next part --------------
An HTML attachment was scrubbed...
URL: </pipermail/twisted-python/attachments/20141008/3a3f10d7/attachment-0002.html>


More information about the Twisted-Python mailing list