HomePhabricator

Mark Owners package reviewers which own nothing in the current diff

Description

Mark Owners package reviewers which own nothing in the current diff

Summary:
Ref PHI91. When Owners (or Herald, or manual user action) adds package reviewers to a revision, later updates to the revision make some of them less relevant or irrelevant.

Provide a hint when a package reviewer doesn't own any of the paths that a diff changes. Humans can then decide if the reviewer is obsolete/irrelevant or not.

This is a rough cut to get the feature working, design could probably use some tweaking if it sticks.

Test Plan:

Screen Shot 2017-09-29 at 1.35.51 PM.png (394×670 px, 53 KB)

Reviewers: amckinley

Reviewed By: amckinley

Subscribers: jboning

Differential Revision: https://secure.phabricator.com/D18663

Details

Provenance
epriestleyAuthored on Sep 29 2017, 8:25 PM
epriestleyPushed on Sep 29 2017, 10:06 PM
Reviewer
amckinley
Differential Revision
D18663: Mark Owners package reviewers which own nothing in the current diff
Parents
rPa39f5e11139a: Correct bad context path when doing pattern search inside a repository
Branches
Unknown
Tags
Unknown
Build Status
Buildable 18591
Build 25041: Run Core Tests