diff options
author | Tomasz Maczukin <tomasz@maczukin.pl> | 2017-03-02 17:44:15 +0100 |
---|---|---|
committer | Tomasz Maczukin <tomasz@maczukin.pl> | 2017-03-02 17:48:00 +0100 |
commit | 5c1aa5fb65ec7474956e6972e40b04b3a967c338 (patch) | |
tree | 9588d71165d0cbc57e56da1c123a201c9d2a5c8e /app/models/ci | |
parent | 1bbf2c2cd16140aa95bbf93368209b16795172bd (diff) | |
download | gitlab-ce-5c1aa5fb65ec7474956e6972e40b04b3a967c338.tar.gz |
Add some fixes and refactoring after review
Diffstat (limited to 'app/models/ci')
-rw-r--r-- | app/models/ci/build.rb | 14 |
1 files changed, 4 insertions, 10 deletions
diff --git a/app/models/ci/build.rb b/app/models/ci/build.rb index 0c0bbb9d41d..5ef089c87c5 100644 --- a/app/models/ci/build.rb +++ b/app/models/ci/build.rb @@ -510,23 +510,17 @@ module Ci def steps [ - Gitlab::Ci::Build::Response::Step.from_commands(self), - Gitlab::Ci::Build::Response::Step.from_after_script(self) + Gitlab::Ci::Build::Step.from_commands(self), + Gitlab::Ci::Build::Step.from_after_script(self) ].compact end def image - image = Gitlab::Ci::Build::Response::Image.new(options[:image]) - return unless image.valid? - image + Gitlab::Ci::Build::Image.from_image(self) end def services - services = options[:services].map do |service| - Gitlab::Ci::Build::Response::Image.new(service) - end - - services.select(&:valid?).compact + Gitlab::Ci::Build::Image.from_services(self) end def artifacts |