1
0
Fork 0
mirror of https://we.phorge.it/source/arcanist.git synced 2025-01-07 21:31:01 +01:00
phorge-arcanist/src/workflow
epriestley dfde57ff81 Improve two error handling behaviors in arc upgrade
Summary:
Fixes T9222. Two issues here:

  - First, we currently continue on error. Throw instead. I just swapped us from "phutil_passthru()" to "execx()" since I don't think printing out the "pulling from remote..." status messages is very important, and this makes it easier to raise a useful exception.
  - Second, if you have a dirty working copy we currently may try to do some sort of silly stuff which won't work, like prompt you to amend changes. Instead, do a slightly lower-level check and just bail.

Test Plan:
  - Ran `arc upgrade` with a dirty working copy and got a tailored, useful error.
  - Ran `arc upgrade` with an artificially bad `git pull` command, got a failure with a specific error message.

Reviewers: chad

Reviewed By: chad

Maniphest Tasks: T9222

Differential Revision: https://secure.phabricator.com/D14317
2015-10-22 19:55:16 +00:00
..
exception pht all the things 2015-05-13 21:00:53 +10:00
ArcanistAliasWorkflow.php pht all the things 2015-05-13 21:00:53 +10:00
ArcanistAmendWorkflow.php pht all the things 2015-05-13 21:00:53 +10:00
ArcanistAnoidWorkflow.php Don't explicitly name abstract base classes 2014-07-22 07:49:15 +10:00
ArcanistBackoutWorkflow.php Minor linter fix 2015-08-08 10:18:59 +10:00
ArcanistBookmarkWorkflow.php Minor linter fixes 2015-05-15 07:09:30 +10:00
ArcanistBranchWorkflow.php Minor linter fixes 2015-05-15 07:09:30 +10:00
ArcanistBrowseWorkflow.php pht all the things 2015-05-13 21:00:53 +10:00
ArcanistCallConduitWorkflow.php Use phutil_json_decode instead of json_decode 2015-05-04 22:36:14 +10:00
ArcanistCloseRevisionWorkflow.php pht all the things 2015-05-13 21:00:53 +10:00
ArcanistCloseWorkflow.php Use PhutilInvalidStateException 2015-06-18 22:41:30 +10:00
ArcanistCommitWorkflow.php phtize a bunch more strings 2015-05-22 17:09:56 +10:00
ArcanistCoverWorkflow.php pht all the things 2015-05-13 21:00:53 +10:00
ArcanistDiffWorkflow.php Add a --target flag to arc unit for uploading results 2015-09-29 09:51:51 -07:00
ArcanistDownloadWorkflow.php Fix some format strings 2015-05-25 21:24:40 +10:00
ArcanistExportWorkflow.php Remove call to "arcanist.projectinfo" from arc export 2015-05-25 18:31:09 +10:00
ArcanistFeatureWorkflow.php Improve arc's handling of dirty submodules in Git 2015-09-21 12:40:06 -07:00
ArcanistFlagWorkflow.php pht all the things 2015-05-13 21:00:53 +10:00
ArcanistGetConfigWorkflow.php pht all the things 2015-05-13 21:00:53 +10:00
ArcanistHelpWorkflow.php phtize a bunch more strings 2015-05-22 17:09:56 +10:00
ArcanistInstallCertificateWorkflow.php pht all the things 2015-05-13 21:00:53 +10:00
ArcanistLandWorkflow.php Fix wrong plural of an arc land message 2015-08-20 04:19:22 -07:00
ArcanistLiberateWorkflow.php Minor linter fixes 2015-05-15 07:09:30 +10:00
ArcanistLintersWorkflow.php arc linters: switch name and short 2015-09-08 10:39:26 -07:00
ArcanistLintWorkflow.php Remove leftover code for "postponed" lint and unit status 2015-08-30 21:56:12 +10:00
ArcanistListWorkflow.php Improve arc's handling of dirty submodules in Git 2015-09-21 12:40:06 -07:00
ArcanistPasteWorkflow.php pht all the things 2015-05-13 21:00:53 +10:00
ArcanistPatchWorkflow.php Fix sanity checks for patch workflow 2015-05-25 18:41:28 +10:00
ArcanistPhrequentWorkflow.php Improve arc's handling of dirty submodules in Git 2015-09-21 12:40:06 -07:00
ArcanistRevertWorkflow.php pht all the things 2015-05-13 21:00:53 +10:00
ArcanistSetConfigWorkflow.php pht all the things 2015-05-13 21:00:53 +10:00
ArcanistShellCompleteWorkflow.php Fix arcanist shell completion 2015-06-19 13:15:44 +10:00
ArcanistStartWorkflow.php Minor tidying of ArcanistPhrequentWorkflow classes 2015-02-01 21:49:53 +11:00
ArcanistStopWorkflow.php Minor tidying of ArcanistPhrequentWorkflow classes 2015-02-01 21:49:53 +11:00
ArcanistTasksWorkflow.php Improve arc's handling of dirty submodules in Git 2015-09-21 12:40:06 -07:00
ArcanistTimeWorkflow.php Minor tidying of ArcanistPhrequentWorkflow classes 2015-02-01 21:49:53 +11:00
ArcanistTodoWorkflow.php pht all the things 2015-05-13 21:00:53 +10:00
ArcanistUnitWorkflow.php Add a --target flag to arc unit for uploading results 2015-09-29 09:51:51 -07:00
ArcanistUpgradeWorkflow.php Improve two error handling behaviors in arc upgrade 2015-10-22 19:55:16 +00:00
ArcanistUploadWorkflow.php Improve temporary file upload API and add viewPolicy support 2015-09-07 12:44:59 -07:00
ArcanistVersionWorkflow.php pht all the things 2015-05-13 21:00:53 +10:00
ArcanistWhichWorkflow.php phtize a bunch more strings 2015-05-22 17:09:56 +10:00
ArcanistWorkflow.php Remove ArcanistWorkflow::setWorkingCopy 2015-10-16 09:53:01 -07:00