On 09:39 pm, j...@mumak.net wrote:
Given Trac, maybe numbering review points + being direct about the change would make it easier to inspect replies, which must in turn mention the numbered points. For this to work, the reviewer would have to clearly distinguish between "I would like you to change X" and "Looking at A inspires the though B."
Here's a thought: every review should have 3 sections. "According to process requirements, you must change XYZ", "I would like you to change XYZ", and "here are some things to think about".
I know that as I respond to a review, I know I'd like to have those things clearly separated out.
_______________________________________________ Twisted-Python mailing list Twisted-Python@twistedmatrix.com http://twistedmatrix.com/cgi-bin/mailman/listinfo/twisted-python
