We should actually give latest success builds as well

e51d4a05 · Lin Jen-Shin · af86b8c2 · e51d4a05
Commit e51d4a05 authored Jul 18, 2016 by Lin Jen-Shin
Hide whitespace changes
Inline Side-by-side

Showing with 1 addition and 1 deletion

app/models/project.rb app/models/project.rb +1 -1

No files found.
--- a/app/models/project.rb
+++ b/app/models/project.rb
@@ -432,7 +432,7 @@ class Project < ActiveRecord::Base
  def latest_success_builds_for(ref = 'HEAD')
    Ci::Build.joins(:pipeline).
      merge(pipelines.where(ref: ref).success.latest).
-      with_artifacts
+      with_artifacts.success.latest
  end
  def merge_base_commit(first_commit_id, second_commit_id)