[Findit] Auto revert might conflict with other changes |
||
Issue descriptionFindit's auto revert might conflict with other changes, for example there might be a depending change of the culprit or a fix to the culprit already committed when Findit is trying to revert the culprit. Solution to it could be: 1. Naive solution: don't commit the revert if there is a change from the culprit author landed. 2. More reliable solution: a. create the revert b. run a try job to verify the revert can pass compile and failed tests passes c. commit the revert if verified; otherwise abandon the revert.
,
Jun 13 2018
Issue 846922 has been merged into this issue.
,
Jun 14 2018
Could we copy miu@'s suggestions here from the duplicated bug 844264 ? I'd prefer to use bug 844264 instead, and update the bug description, but creating this new one is fine too -- just lost some discussion and context.
,
Jul 24
|
||
►
Sign in to add a comment |
||
Comment 1 by chanli@chromium.org
, Jun 13 2018