Ship It should be carried forward if commit content hasn't changed

RESOLVED INVALID

Status

P2
normal
RESOLVED INVALID
4 years ago
2 months ago

People

(Reporter: gps, Unassigned)

Tracking

Details

(Reporter)

Description

4 years ago
If there is a Ship It on an existing review request and someone rebases and pushes again and no file merges have occurred (the content of files is identical), Ship It should be carried forward to the new review request.

This will be useful for reducing bug/review spam when commits are unlanded.
Priority: -- → P2
Why do the file contents have to be the same? If I get "r+ with comments addressed" and I address those comments and re-push the series of patches it should keep the r+. I'm thinking particularly of the workflow in bug 1175166.
See Also: → bug 1175166
Duplicate of this bug: 1179462
Product: Developer Services → MozReview

Comment 3

2 months ago
MozReview is now obsolete. Please use Phabricator instead. Closing this bug.
Status: NEW → RESOLVED
Last Resolved: 2 months ago
Resolution: --- → INVALID
You need to log in before you can comment on or make changes to this bug.