Page MenuHomePhabricator

Improve Drydock errors for empty commits and missing changes

Authored by epriestley on Mar 6 2016, 11:37 PM.



Ref T10093. Show better errors when a commit fails because it has already been merged and when a fetch fails because the ref isn't present in the remote.

Test Plan

Screen Shot 2016-03-06 at 3.27.56 PM.png (595×1 px, 91 KB)

Screen Shot 2016-03-06 at 3.32.13 PM.png (515×1 px, 96 KB)

Diff Detail

rP Phabricator
Lint Not Applicable
Tests Not Applicable

Event Timeline

epriestley retitled this revision from to Improve Drydock errors for empty commits and missing changes.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: chad.

we've been wondering for about a month now why on earth this would happen when no error output is produced

'git fetch %s && '.
'git checkout --force -B %s %s && '.
'git reset --hard %s && '.
'git clean -f -f -d -x',


So that was our "foolproof" method of totally nuking and resetting the contents of a workspace before leasing it out again, and ironically, it does work all of the time, unless there is nothing to change about it.

chad edited edge metadata.


This revision is now accepted and ready to land.Mar 7 2016, 4:32 AM
This revision was automatically updated to reflect the committed changes.