On Thu, Oct 25, 2012 at 04:42:57PM -0400, Derek Buitenhuis wrote:
> On 25/10/2012 2:28 PM, Diego Biurrun wrote:
> > +Give other developers a reasonable amount of time to look at and review
> > +patches before you push them. Not everybody is online 24/7, but may wish
> > +to look at and comment on a patch nonetheless. The time you leave depends
> > +on the urgency and complexity of the patch. Use your common sense to pick
> > +a timeframe that allows everybody that you think may wish to comment
> > +and/or should comment on the change an opportunity to see it.
> 
> Shouldn't anyone who has push access already be acquainted with this?

It's better to write guidelines down than hope for everybody to learn
them through osmosis.  We have had enough incidents lately that served
as inspiration for this patch...

Diego
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to