Awesome work, Shane! What do you think about a dashboard so we can see these over time?
-chris On Tue, May 3, 2016 at 7:23 AM, Shane Tomlinson <[email protected]> wrote: > Hey all! > > 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. > > merged: 1454, unmerged: 229 > > Of those that merged, time to merge: > > mean: 2 days 23 hours 30 minutes 54 seconds > median: 0 days 11 hours 44 minutes 7 seconds > 75%: 2 days 18 hours 7 minutes 58 seconds > 90%: 6 days 22 hours 55 minutes 10 seconds > low: 0 days 0 hours 0 minutes 4 seconds > high: 140 days 2 hours 24 minutes 34 seconds > > The low of 0 days, 0 hours, 0 minutes and 4 seconds. Wow. > 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. > > 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. > > 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. > > Shane > > > > _______________________________________________ > Dev-fxacct mailing list > [email protected] > https://mail.mozilla.org/listinfo/dev-fxacct > >
_______________________________________________ Dev-fxacct mailing list [email protected] https://mail.mozilla.org/listinfo/dev-fxacct
