Improve UI for builds

app/views/builds/_build.html.haml

@@ -8,11 +8,22 @@
 
   %td
     - if build.job
-      = build.job.name
       - if build.for_tag?
-        %span.label.label-primary
-          %i.icon-tag
-          = build.ref
+        Tag
+        ·
+        #{build.ref}
+      - else
+        Commit
+        ·
+        #{build.short_sha}
+  %td
+    = build.job.name
+    - if build.tags.any?
+      .pull-right
+        - build.tag_list.each do |tag|
+          %span.label.label-primary
+            = tag
+
 
   %td.duration
     - if build.duration

app/views/builds/show.html.haml

@@ -36,12 +36,16 @@
   .col-md-9
     .build-head.alert{class: build_status_alert_class(@build)}
       %h4
-        Build for commit
-        %code #{@build.short_sha}
-        from
+        - if @build.for_tag?
+          Build for tag
+          %code #{@build.ref}
+        - else
+          Build for commit
+          %code #{@build.short_sha}
+          from
 
-        = link_to project_path(@build.project, ref: @build.ref) do
-          %span.label.label-primary= "#{@build.ref}"
+          = link_to project_path(@build.project, ref: @build.ref) do
+            %span.label.label-primary= "#{@build.ref}"
 
         - if @build.duration
           .pull-right
@@ -73,6 +77,7 @@
           Test coverage
         %h1 #{@build.coverage}%
 
+
     .build-widget
       %h4.title
         Build
@@ -118,6 +123,14 @@
         %span.attr-name Message:
         #{@build.commit.git_commit_message}
 
+    - if @build.tags.any?
+      .build-widget
+        %h4.title
+          Tags
+        - @build.tag_list.each do |tag|
+          %span.label.label-primary
+            = tag
+
     - if @builds.present?
       .build-widget
         %h4.title #{pluralize(@builds.count, "other build")} for #{@build.short_sha}:

app/views/commits/show.html.haml

@@ -57,6 +57,7 @@
     %tr
       %th Status
       %th Build ID
+      %th Trigger
       %th Job
       %th Duration
       %th Finished at
@@ -74,6 +75,7 @@
       %tr
         %th Status
         %th Build ID
+        %th Trigger
         %th Job
         %th Duration
         %th Finished at

spec/features/commits_spec.rb

@@ -5,7 +5,8 @@
     login_as :user
     @project = FactoryGirl.create :project
     @commit = FactoryGirl.create :commit, project: @project
-    @build = FactoryGirl.create :build, commit: @commit
+    @job = FactoryGirl.create :job, project: @project
+    @build = FactoryGirl.create :build, commit: @commit, job: @job
   end
 
   describe "GET /:project/commits/:sha" do

spec/requests/api/builds_spec.rb

@@ -17,6 +17,7 @@
     describe "POST /builds/register" do
       it "should start a build" do
         commit = FactoryGirl.create(:commit, project: project)
+        job = FactoryGirl.create :job, project: project
         build = commit.create_builds.first
 
         post api("/builds/register"), token: runner.token