Patrick, It shouldn't have a significant impact on any new contributers since the reviews of their patches would could start as soon as the patch is posted. I'm interested in your experience in this area - the only other project I have worked on with a cool-off period is Hive, and a cool-off does not seem to have had a negative impact on community growth. Do you know how other projects address the problem of giving larger portion of the community a chance to review a chance prior to commit?
Thanks, Lenni On Tue, Nov 10, 2015 at 5:34 PM, Patrick Hunt <[email protected]> wrote: > Ugh. I personally dislike the idea of any cool-off. It's hard enough for > projects to get folks to review changes, introducing some artificial > "freeze" is not going to be a pain and turn people off. > > Patrick > > On Tue, Nov 10, 2015 at 4:37 PM, Xuefu Zhang <[email protected]> wrote: > > > I think 24h is usually enough. It gets complicated when we start > > considering weekends, holidays (different countries, regions), etc. > Longer > > waiting period is at the discretion of the one who's going to make the > > commit. > > > > Thanks, > > Xuefu > > > > On Tue, Nov 10, 2015 at 4:25 PM, Ma, Junjie <[email protected]> wrote: > > > > > +1 cool off 48 hrs because of the weekend. > > > > > > Best regards, > > > > > > Colin Ma(Ma Jun Jie) > > > > > > -----Original Message----- > > > From: Lenni Kuff [mailto:[email protected]] > > > Sent: Wednesday, November 11, 2015 3:25 AM > > > To: [email protected] > > > Subject: [DISCUSS] Cool off period for commits? > > > > > > Currently Sentry has not policy in place for a cool off period for > > commits > > > (time after patch has gotten +1'ed that the change can be committed). > > This > > > limits the opportunity other people in the community can review a > change > > > prior to it going in. This is particularly important since we have > > > committers across many different time zones > > > > > > What do you all think about adding a cool-off period for all commits > > after > > > a patch has gotten a +1? The Hive project uses 24 hours, so we could go > > > with that. Could also use something longer like 48 or 72 hours. > Thoughts? > > > > > > Thanks, > > > Lenni > > > > > >
