>
>
> > 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.

Reply via email to