Closed
Bug 1021945
Opened 10 years ago
Closed 10 years ago
Don't create bug attachments for child requests, but do mirror reviews
Categories
(MozReview Graveyard :: General, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: mcote, Unassigned)
References
Details
When the hg review-repo hook creates review requests, it creates one parent review request containing a squashed diff of all commits to be reviewed and then one child review request for each commit.
Since the parent contains all the changes, it should be the only one to create an associated attachment in the bug--when it is r+ed, the patch in its entirety (all commits) are ready to be landed. However, child requests should still have their reviews mirrored over, since reviewers might be focussing on individual commits.
Thus all requests should have the associated bug ID, and all reviews to all requests in the "family" should be mirrored to that bug, but any request with a parent should not post an attachment to the bug.
Comment 1•10 years ago
|
||
George, are you interested in taking this?
Flags: needinfo?(george.miroshnykov)
Comment 2•10 years ago
|
||
I believe Mark is already working on this in rbbz, not much I can do inside the hook.
But if there's something I can help with, please let me know!
Flags: needinfo?(george.miroshnykov)
Comment 3•10 years ago
|
||
I see, sounds good!
Assignee | ||
Comment 4•10 years ago
|
||
Yup, I'm going to handle this. Should be pretty simple.
Assignee: nobody → mcote
Status: NEW → ASSIGNED
Assignee | ||
Comment 5•10 years ago
|
||
I committed this (https://github.com/mozilla/rbbz/commit/4c5d78bfba1b6fd8018239f91ac26f75c397e494) but just realized I got it backwards. The parent reviews, the ones we want created in Bugzilla, have the "depends on" field set, not the children. It would be easy enough to switch, except it would break the work flow for anyone not using a review repo, since they wouldn't normally use depends on. I'll have to think a bit about this.
Assignee | ||
Comment 6•10 years ago
|
||
This was fixed in a different way in http://hg.mozilla.org/hgcustom/version-control-tools/rev/d9c4a5eef14e by checking if the review request is squashed or not.
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Updated•10 years ago
|
Product: bugzilla.mozilla.org → Developer Services
Updated•9 years ago
|
Product: Developer Services → MozReview
You need to log in
before you can comment on or make changes to this bug.
Description
•