Fixing early return.

config/build.yml

@@ -44,14 +44,15 @@ cm:
     key: sync
     # A different config key is used by the deploy:update step, which is executed on Acquia Cloud.
     # This must also have a corresponding value at cm.core.dirs.vcs.
+    # If you are NOT on Acquia hosting, you probably need to change this to 'sync'.
     # @see https://github.com/acquia/blt/issues/678
     deploy-key: vcs
     dirs:
       # Corresponding value is defined in config.settings.php.
       sync:
         path: ${cm.core.path}/default
       vcs:
-        path: ${cm.core.path}/default
+        path: ${cm.core.dirs.sync.path}
   features:
     no-overrides: true
 

scripts/blt/ci/internal/create_blt_project.sh

@@ -27,6 +27,9 @@ yaml-cli update:value blt/project.yml project.local.hostname '127.0.0.1:8888'
 # Define BLT's deployment endpoints.
 yaml-cli update:value blt/project.yml git.remotes.0 bolt8@svn-5223.devcloud.hosting.acquia.com:bolt8.git
 yaml-cli update:value blt/project.yml git.remotes.1 git@github.com:acquia-pso/blted8.git
+# Set cm.core.deploy-key to sync since we are not executing this on Acquia Cloud.
+touch blt/project.local.yml
+yaml-cli update:value blt/project.local.yml cm.core.deploy-key sync
 
 git add -A
 git commit -m 'Adding new dependencies from BLT update.' -n

src/Robo/BltTasks.php

@@ -89,9 +89,9 @@ protected function invokeCommand($command_name, array $args = []) {
       $exit_code = $application->runCommand($command, $input, $this->output());
       $this->invokeDepth--;
 
-      // The application will catch any exceptions thrown in the run command.
-      // We must check the exit code and throw our own exception. This allows
-      // us to do without checking the exit code of every invoked command.
+      // The application will catch any exceptions thrown in the executed
+      // command. We must check the exit code and throw our own exception. This
+      // obviates the need to check the exit code of every invoked command.
       if ($exit_code) {
         throw new BltException("Command `$command_name {$input->__toString()}` exited with code $exit_code.");
       }

src/Robo/Commands/Deploy/DeployCommand.php

@@ -560,7 +560,11 @@ public function updateSites() {
     // /config/default. ACE internally sets the vcs configuration
     // directory to /config/default, so we use that.
     // @see https://github.com/acquia/blt/issues/678
+    if (!isset($_ENV['AH_SITE_ENVIRONMENT']) && $this->getConfigValue('cm.core.deploy-key') == 'vcs') {
+      $this->logger->warning('cm.core.deploy-key is set to "vcs". This will only work on Acquia Cloud. If you are not using Acquia Cloud, change this configuration value.');
+    }
     $this->config->set('cm.core.key', $this->getConfigValue('cm.core.deploy-key'));
+
     // Disable alias since we are targeting specific uri.
     $this->config->set('drush.alias', '');
 

src/Robo/Commands/Setup/ConfigCommand.php

@@ -36,11 +36,12 @@ public function import() {
       // required config is exported.
       if (in_array($strategy, ['core-only', 'config-split'])) {
         $core_config_file = $this->getConfigValue('docroot') . '/' . $this->getConfigValue("cm.core.dirs.$cm_core_key.path") . '/core.extension.yml';
+
         if (!file_exists($core_config_file)) {
           $this->logger->warning("BLT will NOT import configuration, $core_config_file was not found.");
+          // This is not considered a failure.
+          return 0;
         }
-        // This is not considered a failure.
-        return 0;
       }
 
       $task = $this->taskDrush()