> > > > But you can't turn around and add another feature on a ticket that >> already >> > has positive review. >> > Not another feature, sure. But fixing an error, even a typo (especially if it's a misleading typo like forgetting the word "not") sounds like adequate grounds to revert a positive review. Our testing process should accommodate quality, not vice versa.
> Testing imposes a significant delay between merging into the develop > branch and closing the ticket. > My understanding of git is of course weaker than anyone else's, but ... isn't the whole POINT of having a DVCS so that one can test a change and do whatever one wants before actually merging into the develop branch? Again, I would argue that it is far less likely for someone to respond to even a fairly major bug in a "new" ticket than to do a quick fix so that their contribution "gets in" with a ticket that is so so so close to being merged. -- You received this message because you are subscribed to the Google Groups "sage-devel" group. To unsubscribe from this group and stop receiving emails from it, send an email to sage-devel+unsubscr...@googlegroups.com. To post to this group, send email to sage-devel@googlegroups.com. Visit this group at http://groups.google.com/group/sage-devel. For more options, visit https://groups.google.com/d/optout.