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

RESOLVED INVALID

Status

--
enhancement
RESOLVED INVALID
2 years ago
2 years ago

People

(Reporter: xidorn, Unassigned)

Tracking

Details

(Reporter)

Description

2 years ago
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.
(Reporter)

Updated

2 years ago
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
Last Resolved: 2 years ago
Resolution: --- → INVALID
You need to log in before you can comment on or make changes to this bug.