summaryrefslogtreecommitdiff
path: root/app/models/ci
diff options
context:
space:
mode:
authorTomasz Maczukin <tomasz@maczukin.pl>2017-03-02 17:44:15 +0100
committerTomasz Maczukin <tomasz@maczukin.pl>2017-03-02 17:48:00 +0100
commit5c1aa5fb65ec7474956e6972e40b04b3a967c338 (patch)
tree9588d71165d0cbc57e56da1c123a201c9d2a5c8e /app/models/ci
parent1bbf2c2cd16140aa95bbf93368209b16795172bd (diff)
downloadgitlab-ce-5c1aa5fb65ec7474956e6972e40b04b3a967c338.tar.gz
Add some fixes and refactoring after review
Diffstat (limited to 'app/models/ci')
-rw-r--r--app/models/ci/build.rb14
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