This one is a bit hard to properly explain so I give an example: https://reviewboard.mozilla.org/r/34327/diff/3-6/ I pushed a review (5 at first to be exact). One review P2 had a commit-id CXqGoq9Opq0 I pushed several updated revisions of the code, all with the same commit-id. I then rewrote the entire stack, moved it to 4 commits instead. And P2 became P3. As I wanted to have a complete new review for that one, I changed the commit-id and requested a review for it. Yet, mozreview carried the r+ from earlier on. to test, I re-modified the commit-id and push again. Yet mozreview shows the same review as having 6 different revisions. It shouldn't. It should have treated this review as an entirely new one and re-requested review to the intended person.
I /think/ part of the problem here is we recycled an old review request that had a different MozReview-Commit-ID. We should almost certainly not recycle a review request if it has a different MozReview-Commit-ID because if the IDs are different, that means the commit logically changed and a new review request should be created.
Product: Developer Services → MozReview
MozReview is now obsolete. Please use Phabricator instead. Closing this bug.
Status: NEW → RESOLVED
Last Resolved: 6 months ago
Resolution: --- → INVALID
You need to log in before you can comment on or make changes to this bug.