+1 What do you think about assigning someone to each PR to make sure it gets through the process? We don't currently seem to be using github's "assignee" field. Committers can make themselves the assignee, but for contributors we could assign a committer who will make sure the PR gets reviewed and merged in a timely fashion.
-Dan On Tue, Jul 9, 2019 at 10:34 AM Mark Hanson <mhan...@pivotal.io> wrote: > Hi All, > > TL;DR > > Can we have a norm( preferred, but not required ) of providing feedback > within seven days of the last checkin to a PR? > > Long version > > I have just spent a bit of time reviewing PRs that have been open for a > while and sent some emails to reviewers of the ones that are open the > longest. In my humble opinion, it would be very nice if we could close out > some of the older PRs where the requester has made changes to, but > reviewers have not re-reviewed. An ideal norm would seem to be 7 days. One > might notice that I have a PR that I requested a change on, that I have not > provided feedback on, so I am in the same boat... > > Thoughts? > > Thanks, > Mark
