Just detect exit status rather than check ref

Feedback: https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/13416#note_37193731 So we just try a cheaper way to detect it if it works or not

Just detect exit status rather than check ref
023d6749 · Lin Jen-Shin · 412db187 · 023d6749
Commit 023d6749 authored Aug 10, 2017 by Lin Jen-Shin
Show whitespace changes
Inline Side-by-side

Showing with 1 addition and 1 deletion

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

No files found.
--- a/app/models/repository.rb
+++ b/app/models/repository.rb
@@ -1024,7 +1024,7 @@ class Repository
    message, status = run_git(args)

    # Make sure ref was created, and raise Rugged::ReferenceError when not
-    raise Rugged::ReferenceError, message unless ref_exists?(target_ref)
+    raise Rugged::ReferenceError, message if status != 0
  end

  def create_ref(ref, ref_path)