Closed Bug 1275130 Opened 9 years ago Closed 9 years ago

Consider adding a warning / prompt when unrelated patch is included in a revew push

Categories

(MozReview Graveyard :: Integration: Mercurial, enhancement)

Product:
MozReview Graveyard
Component:
Integration: Mercurial
Type:
enhancement
Priority:
Not set
Severity:
normal

Tracking

(Not tracked)

Status:
RESOLVED INVALID

People

(Reporter: xidorn, Unassigned)

Details

I accdentally pushed an unrelated patch to MozReview today in bug 1274520, which makes me think it might be good to have an additional check before pushing that every commit included are related to the bug. This could probably be an [Yn] question so that people could still push if they intend to do so.
Severity: normal → enhancement
This is what the draft stage is for. By default we will prompt if you'd like to publish the review series (and just before this prompt we output all the commit information for the review requests we created). If you do not publish, you can repush the proper changes and all will be fine, or even discard the entire review series and it will not be made public.
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → INVALID
You need to log in before you can comment on or make changes to this bug.