From e34bdf6c74f181dbe4f36157baac80e64442097f Mon Sep 17 00:00:00 2001 From: epriestley Date: Fri, 23 May 2014 11:00:07 -0700 Subject: [PATCH] Fix diffusion.looksoon call to actually work Summary: This is all derped up, and we ignore the error so old installs don't break. Underp it. Test Plan: Did `arc land`, verified that the call actually went through successfully. Reviewers: btrahan, davedash Reviewed By: davedash Subscribers: epriestley Differential Revision: https://secure.phabricator.com/D9271 --- src/workflow/ArcanistLandWorkflow.php | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/workflow/ArcanistLandWorkflow.php b/src/workflow/ArcanistLandWorkflow.php index 55df534c..6591a930 100644 --- a/src/workflow/ArcanistLandWorkflow.php +++ b/src/workflow/ArcanistLandWorkflow.php @@ -191,7 +191,7 @@ EOTEXT if ($this->useSquash) { $this->rebase(); - $this->squash(); + $this->squash(); } else { $this->merge(); } @@ -1015,7 +1015,7 @@ EOTEXT $this->getConduit()->callMethodSynchronous( 'diffusion.looksoon', array( - 'callsign' => $this->getRepositoryCallsign(), + 'callsigns' => array($this->getRepositoryCallsign()), )); } catch (ConduitClientException $ex) { // If we hit an exception, just ignore it. Likely, we are running