Closed
Bug 1194354
Opened 8 years ago
Closed 5 years ago
Add "Notes to Reviewer" Field
Categories
(MozReview Graveyard :: General, defect, P3)
Tracking
(Not tracked)
RESOLVED
INVALID
People
(Reporter: dminor, Unassigned)
Details
We're planning to make the description field contain just the commit summary in Bug 1123139 to encourage people to write meaningful commit messages. One use case for the description field is to indicate that this is initial work and that the review requestor is looking for feedback rather than a detailed review. This can be done in the commit message and then the commit message can be amended when the patch is updated. Another use case is asking a reviewer to look at one section of a commit and a different reviewer to look at another section of a commit. This could be handled in the commit message or by splitting the commit into two, but it seems that it would be nice to have "Notes to Reviewer" field for this case.
Reporter | ||
Comment 1•8 years ago
|
||
The plan is for this to be available only on the parent review request for now.
Assignee | ||
Updated•8 years ago
|
Product: Developer Services → MozReview
Comment 3•8 years ago
|
||
I think this should actually be available on all commits.
Comment 4•8 years ago
|
||
Duping this to a newer bug that has a bit more context.
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → DUPLICATE
Comment 5•8 years ago
|
||
I'm a bit confused. This bug is about the patch author being able to write the reviewer a supplementary note. Bug 1248008 seems to be about the reviewer being able to write a review comment for the commit message.
Comment 6•8 years ago
|
||
Oh you're right. I thought for sure this had been filed before, but I can't find it. Huh.
Status: RESOLVED → REOPENED
Resolution: DUPLICATE → ---
Comment 7•5 years ago
|
||
MozReview is now obsolete. Please use Phabricator instead. Closing this bug.
Status: REOPENED → RESOLVED
Closed: 8 years ago → 5 years ago
Resolution: --- → INVALID
You need to log in
before you can comment on or make changes to this bug.
Description
•