Stale PR stats @Jun01
Mark J Cox
mark at openssl.org
Tue Jun 2 14:34:49 UTC 2020
Rich Salz mailed me about the issues that were in the "waiting for
reporter" state in my summary noting that some of them shouldn't be in
that state. I investigated just now and made some changes to my
script, so if you are a reviewer or reporter you may get some extra
pings today as things moved to the right state.
The script looked at the github timeline 'reviewed' events, however
those events only happen when changes are requested or the review is
approved or rejected. So if a reporter does an update and re-requests
a review, it still shows up for the script in state
"changes_requested" which isn't true. So now the script also looks at
"review_requested" events, and a "review_requested" will change the
state from "changes_requested".
When looking through the list one by one I spotted quite a few PRs
which had changes_requested but where the reporter had made some
changes or commented about the changes but had not re-requested a
review. I manually did that re-request.
I'll also add a OpenSSL Machine comment to stale "changes_requested"
state issues to remind the reporter to do that action (and to be sure
to re-request a review once they have done so).
So here was the state before:
> waiting for reporter ( 16 issues, median 299.5 days)
>
> 11530 branch: 1.1.1, branch: master, cla: trivial,
> reviewed:changes_requested days:46
> 11514 reviewed:changes_requested days:50
> 11310 reviewed:changes_requested days:74
> 10724 reviewed:changes_requested days:37
> 10590 reviewed:changes_requested days:146
> 9575 reviewed:changes_requested days:293
> 9461 reviewed:changes_requested days:306
> 9427 reviewed:changes_requested days:313
> 9243 reviewed:changes_requested days:315
> 9240 reviewed:changes_requested days:340
> 8992 reviewed:changes_requested days:257
> 8962 reviewed:changes_requested days:75
> 8730 reviewed:changes_requested days:354
> 8674 reviewed:changes_requested days:409
> 7961 reviewed:changes_requested days:511
> 7432 reviewed:changes_requested days:590
>
> waiting for review ( 2 issues, median 40.5 days)
>
> 11526 approval: review pending, branch: 1.1.1, branch: master,
> reviewed:approved days:32
> 11278 approval: review pending, branch: master, reviewed:commented days:49
And now it is:
waiting for reporter ( 9 issues, median 308 days)
11530 branch: 1.1.1, branch: master, cla: trivial,
reviewed:changes_requested days:48
10590 reviewed:changes_requested days:148
9575 reviewed:changes_requested days:294
9461 reviewed:changes_requested days:308
9427 reviewed:changes_requested days:315
8962 reviewed:changes_requested days:76
8674 reviewed:changes_requested days:411
7961 reviewed:changes_requested days:513
7432 reviewed:changes_requested days:592
waiting for review ( 8 issues, median 51.5 days)
11526 approval: review pending, branch: 1.1.1, branch: master,
reviewed:approved days:34
11514 reviewed:review pending days:52
11278 approval: review pending, branch: master, reviewed:commented days:51
10724 reviewed:review pending days:38
10632 reviewed:review pending days:35
8992 reviewed:review pending days:258
8730 reviewed:review pending days:356
6725 milestone:Assessed, reviewed:review pending days:363
More information about the openssl-project
mailing list