[Gluster-infra] [Bug 1451184] Assign reviewers based on who touched the file last

bugzilla at redhat.com bugzilla at redhat.com
Tue May 16 11:00:53 UTC 2017


Prashanth Pai <ppai at redhat.com> changed:

           What    |Removed                     |Added
                 CC|                            |ppai at redhat.com

--- Comment #2 from Prashanth Pai <ppai at redhat.com> ---
Sometimes, there are changes from new contributors which go unnoticed because
no reviewers were added. I've seen Jeff and Amar diligently going through these
new changes without reviewers. There should be a better mechanism for
maintainers to subscribe or notice new incoming changes and this will greatly

Some thoughts on this:

* Github 'suggests' reviewers for PRs based on git blame. The PR author can
choose or ignore to add them as reviewers. If Gerrit can suggest reviewers in
the UI, it'd be great.

* If Gerrit allows, you could check if the potential reviewer (based on git
blame) has had reviewing activity in recent past (to filter out non active
contributors), and then add them as reviewer.

You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=13q3rARj3d&a=cc_unsubscribe

More information about the Gluster-infra mailing list