On Thu, Jul 19, 2012 at 11:52 PM, Antoine Musso <[email protected]> wrote: > How are the blind spot? You can diff each patchset with its previous one > (though a rebase kind of screw it up). > > Whenever I send a new patchset, I try to add a cover message introducing > what was done in the new patchset so people receive a mail notification > explaining what has been done. > Yes, Gerrit handles this well (except for post-merge fixups, which aren't handled well), and from what I've heard Github has improved and now does a decent job as well. But I've seen other systems (such as Barkeep) where the developers have massively underestimated the complexity that fixups bring to pre-merge review.
Roan _______________________________________________ Wikitech-l mailing list [email protected] https://lists.wikimedia.org/mailman/listinfo/wikitech-l
