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
