Skip to content

Commit

Permalink
Making requirement for composer.lock update after BLT update more clear.
Browse files Browse the repository at this point in the history
  • Loading branch information
grasmash committed Dec 5, 2016
1 parent b55e1b4 commit 3a62ce0
Showing 1 changed file with 4 additions and 3 deletions.
7 changes: 4 additions & 3 deletions src/Composer/Plugin.php
Original file line number Diff line number Diff line change
Expand Up @@ -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('<comment>This may have modified your composer.json and require a subsequent `composer update`</comment>');

// @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('<error>Your composer.json dependencies were modified, you MUST run "composer update" to update your composer.lock file. This is not an error.</error>');
}
}
else {
$this->io->write('<comment>Skipping update of BLT templated files</comment>');
Expand Down

0 comments on commit 3a62ce0

Please sign in to comment.