[PATCH] D46192: Script to match open Phabricator reviews with potential reviewers

Kristof Beyls via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue May 1 06:45:23 PDT 2018


kristof.beyls added a comment.

In https://reviews.llvm.org/D46192#1083794, @javed.absar wrote:

> Hi Kristof. Thanks for this.
>
> One more heuristic that might help getting quick reviews:
>
> 3. If there is overlap between the files touched by the patch-under-review and and a 'reviewer' of recent changes to same file, that person may be a good reviewer.
>
>   This heuristic is helpful where original implementor may have moved on, but new active reviewers in same area are happy to review. Some commit messages have 'reviewers' listed and that could be used.


Interesting idea, thanks for sharing, Javed!
I'll need a bit of time to think on how this could be best implemented.

Thanks!

Kristof


https://reviews.llvm.org/D46192





More information about the llvm-commits mailing list