Merge pull request #113 from SimonHeimberg/plugin_exceptions
Plugin exceptions
This commit is contained in:
commit
eaec65509b
1 changed files with 7 additions and 2 deletions
|
|
@ -52,7 +52,7 @@ class Builder implements LoggerAwareInterface
|
|||
/**
|
||||
* @var array
|
||||
*/
|
||||
protected $config;
|
||||
protected $config = [];
|
||||
|
||||
/**
|
||||
* @var string
|
||||
|
|
@ -217,7 +217,13 @@ class Builder implements LoggerAwareInterface
|
|||
} else {
|
||||
$this->build->setStatus(Build::STATUS_FAILED);
|
||||
}
|
||||
} catch (\Exception $ex) {
|
||||
$success = false;
|
||||
$this->build->setStatus(Build::STATUS_FAILED);
|
||||
$this->buildLogger->logFailure('Exception: ' . $ex->getMessage(), $ex);
|
||||
}
|
||||
|
||||
try {
|
||||
if ($success) {
|
||||
$this->pluginExecutor->executePlugins($this->config, Build::STAGE_SUCCESS);
|
||||
|
||||
|
|
@ -232,7 +238,6 @@ class Builder implements LoggerAwareInterface
|
|||
}
|
||||
}
|
||||
} catch (\Exception $ex) {
|
||||
$this->build->setStatus(Build::STATUS_FAILED);
|
||||
$this->buildLogger->logFailure('Exception: ' . $ex->getMessage(), $ex);
|
||||
}
|
||||
|
||||
|
|
|
|||
Loading…
Add table
Add a link
Reference in a new issue