| Issue 1187: | Allow reviewer to be tagged with a reason for being added as a reviewer | |
| 10 people starred this issue and may be notified of changes. | Back to list |
It would be nice to be able to tag each assigned reviewer with a review priority, such as: - FYI only; no actual expectation of review - Review is considered optional for given reviewer - Review is considered mandatory for given reviewer (with potential interlocks to prevent code from being submitted without review from this reviewer) The concept could be extended with separate, configurable, stackable tags to indicate why the reviewer might have an interest in performing the review, such as: - Code the reviewer directly owns is being modified - Reviewer has specific, helpful expertise in some aspect of the code (such as how to implement a tricky algorithm) - Reviewer is a gatekeeper of code being admitted to a repo - etc. In our organization, lots of reviewers are added to code reviews for all the above cited reasons (and more), but without follow-up notes, it is difficult to convey expectations, priorities, etc. Reviewers can even be confused why they were asked to review in the first place.
Jul 16, 2015
#1
namrata....@gmail.com
Jul 16, 2015
Also, once this is implemented, there should be a way to categorize your dashboard based on your role. E.g. a section for all patches for which you are primary reviewer, a separate section for all patches where you are the verifier, and so on. |
|
| ► Sign in to add a comment |