[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