Making it easy for the reviewer is definitely a bonus of not folding. 
 Another reason not to fold is if the patches have different authors. 
 Finally, if some of the patches change things in earlier patches in direct 
response to reviewer comments, folding them can make it hard to trace 
whether the problem was actually fixed.

However, in some cases there is a sort of patcherrhea and folding them 
together is wise, so think about it on a case by case basis.  What would 
*you* want as a reviewer or release manager for your patch, on specific 
ticket X?

- kcrisman

-- 
You received this message because you are subscribed to the Google Groups 
"sage-devel" group.
To post to this group, send email to sage-devel@googlegroups.com.
To unsubscribe from this group, send email to 
sage-devel+unsubscr...@googlegroups.com.
Visit this group at http://groups.google.com/group/sage-devel?hl=en.


Reply via email to