On Tuesday, April 21, 2015 at 2:11:43 PM UTC-5, Andrew Halberstadt wrote: > On 21/04/15 03:02 PM, Aaron Klotz wrote: > > On 4/21/2015 12:50 PM, Andrew Halberstadt wrote: > >> > >> This could be effective, but if not implemented with care it could > >> also be very de-motivating, especially for a well-intentioned > >> contributor. > > > > Is this really an issue though, given the time and effort required to > > earn sufficient commit access to push to inbound? > > It's the patch author that is on the hook for bustage, not the pusher. > Any contributor can land a patch on inbound by setting 'checkin-needed' > on the bug. > > But contributors aside, it could be de-motivating for employees too. If > I break inbound, I already feel really bad about it.. no need to rub it > in my face :).
I think we're being bit too sensitive here, I'm sure we can all handle a little public shaming on stuff like this. :) If you find yourself on the top of a list like list, and you feel a bit bad about it, good. Learn from it, push to try more often. That's the whole point. _______________________________________________ dev-platform mailing list dev-platform@lists.mozilla.org https://lists.mozilla.org/listinfo/dev-platform
