[FIX] Running Complete stage even on Exception catch. (#1186)
This commit is contained in:
parent
49db1a26ba
commit
db93f55427
|
@ -213,8 +213,6 @@ class Builder implements LoggerAwareInterface
|
||||||
$this->build->setStatus(Build::STATUS_FAILED);
|
$this->build->setStatus(Build::STATUS_FAILED);
|
||||||
}
|
}
|
||||||
|
|
||||||
// Complete stage plugins are always run
|
|
||||||
$this->pluginExecutor->executePlugins($this->config, 'complete');
|
|
||||||
|
|
||||||
if ($success) {
|
if ($success) {
|
||||||
$this->pluginExecutor->executePlugins($this->config, 'success');
|
$this->pluginExecutor->executePlugins($this->config, 'success');
|
||||||
|
@ -236,6 +234,9 @@ class Builder implements LoggerAwareInterface
|
||||||
} catch (\Exception $ex) {
|
} catch (\Exception $ex) {
|
||||||
$this->build->setStatus(Build::STATUS_FAILED);
|
$this->build->setStatus(Build::STATUS_FAILED);
|
||||||
$this->buildLogger->logFailure(Lang::get('exception') . $ex->getMessage());
|
$this->buildLogger->logFailure(Lang::get('exception') . $ex->getMessage());
|
||||||
|
}finally{
|
||||||
|
// Complete stage plugins are always run
|
||||||
|
$this->pluginExecutor->executePlugins($this->config, 'complete');
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|
||||||
|
|
Loading…
Reference in a new issue