<div dir="ltr">Awesome work, Shane!<div><br></div><div>What do you think about a dashboard so we can see these over time?</div><div><br></div><div>-chris</div><div><br></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Tue, May 3, 2016 at 7:23 AM, Shane Tomlinson <span dir="ltr"><<a href="mailto:stomlinson@mozilla.com" target="_blank">stomlinson@mozilla.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div><div><div><div><div><div>Hey all!<br><br></div>It was a long weekend and I felt like hacking on something a bit different, so I wrote a script to generate some stats about content server pull requests.<br><br>merged: 1454, unmerged: 229<br><br>Of those that merged, time to merge:<br><br>mean: 2 days 23 hours 30 minutes 54 seconds<br>median: 0 days 11 hours 44 minutes 7 seconds<br>75%: 2 days 18 hours 7 minutes 58 seconds<br>90%: 6 days 22 hours 55 minutes 10 seconds<br>low: 0 days 0 hours 0 minutes 4 seconds<br>high: 140 days 2 hours 24 minutes 34 seconds<br><br></div>The low of 0 days, 0 hours, 0 minutes and 4 seconds. Wow.<br></div>The high of 140 days, 2 hours, 24 minutes, 34 seconds, I believe that was the massive settings refactor, but I haven't gone back and verified.<br><br></div>A median PR existence of ~ 12 hours is respectable, in general we try to review/merge quickly with the purpose of helping contributors feel valued. Even a 75th percentile of 2 days, 18 hours is pretty alright. After that there is a big long tail, from 75% to 90% jumps to nearly 7 days.  Some PRs just take a long time, and I think this is reflected in a mean that is higher than the 75th percentile.<br><br></div>My intent is to generate these on a bi-weekly basis to help us gauge turn around time. I also want to add time to first comment, with a theory that contributors (external and internal) end up happier and more engaged when they feel their contributions are valued and reviewed quickly.<span class="HOEnZb"><font color="#888888"><br><br></font></span></div><span class="HOEnZb"><font color="#888888">Shane<br><div><div><div><div><br><br></div></div></div></div></font></span></div>
<br>_______________________________________________<br>
Dev-fxacct mailing list<br>
<a href="mailto:Dev-fxacct@mozilla.org">Dev-fxacct@mozilla.org</a><br>
<a href="https://mail.mozilla.org/listinfo/dev-fxacct" rel="noreferrer" target="_blank">https://mail.mozilla.org/listinfo/dev-fxacct</a><br>
<br></blockquote></div><br></div>