Check for keep-around commits once if target and source projects are the same

This reduces the number of RefExists RPC call.

Check for keep-around commits once if target and source projects are the same
48568a9d · Ahmad Sherif · 826105df · 48568a9d
Commit 48568a9d authored Feb 07, 2018 by Ahmad Sherif
Hide whitespace changes
Inline Side-by-side

Showing with 1 addition and 1 deletion

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

No files found.
--- a/app/models/merge_request_diff.rb
+++ b/app/models/merge_request_diff.rb
@@ -290,7 +290,7 @@ class MergeRequestDiff < ActiveRecord::Base
  end

  def keep_around_commits
-    [repository, merge_request.source_project.repository].each do |repo|
+    [repository, merge_request.source_project.repository].uniq.each do |repo|
      repo.keep_around(start_commit_sha)
      repo.keep_around(head_commit_sha)
      repo.keep_around(base_commit_sha)