From 3a62ce0640c169d9c305976c8e215c3a41cf4b8d Mon Sep 17 00:00:00 2001 From: Matthew Grasmick Date: Mon, 5 Dec 2016 11:54:14 -0500 Subject: [PATCH] Making requirement for composer.lock update after BLT update more clear. --- src/Composer/Plugin.php | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/src/Composer/Plugin.php b/src/Composer/Plugin.php index c93e36833..5210d6cd0 100644 --- a/src/Composer/Plugin.php +++ b/src/Composer/Plugin.php @@ -133,10 +133,11 @@ protected function executeBltUpdate($version) { // Rsyncs, updates composer.json, project.yml, executes scripted updates for version delta. $this->executeCommand('blt update'); - $this->io->write('This may have modified your composer.json and require a subsequent `composer update`'); - // @todo check if require or require-dev changed. If so, run `composer update`. - // @todo if require and require-dev did not change, but something else in composer.json changed, execute `composer update --lock`. + $output = $this->executeCommand('composer validate --no-check-all'); + if (strstr($output, 'The lock file is not up to date')) { + $this->io->write('Your composer.json dependencies were modified, you MUST run "composer update" to update your composer.lock file. This is not an error.'); + } } else { $this->io->write('Skipping update of BLT templated files');