Browse Source

Merge pull request #293 from natechicago/292-bugfix

292: Notification email reports SUCCESS even if deployment failed
1.0
Andrés Montañez 8 years ago committed by GitHub
parent
commit
62c575f5b5
  1. 2
      Mage/Command/BuiltIn/DeployCommand.php

2
Mage/Command/BuiltIn/DeployCommand.php

@ -215,7 +215,7 @@ class DeployCommand extends AbstractCommand implements RequiresEnvironment
Console::output('Total time: <bold>' . $timeText . '</bold>.', 1, 2); Console::output('Total time: <bold>' . $timeText . '</bold>.', 1, 2);
// Send Notifications // Send Notifications
$this->sendNotification(self::$failedTasks > 0 ? false : true); $this->sendNotification(self::$deployStatus === self::SUCCEDED);
// Unlock // Unlock
if (file_exists(getcwd() . '/.mage/~working.lock')) { if (file_exists(getcwd() . '/.mage/~working.lock')) {

Loading…
Cancel
Save