Merge branch 'fix/find-commit-with-empty-rev' into 'master'

Keep note commit around if commit_id is present Closes gitaly#1054 See merge request gitlab-org/gitlab-ce!17496

Merge branch 'fix/find-commit-with-empty-rev' into 'master'
b82325cb · Robert Speicher · 0fc80c54 · 2cc43aaa · b82325cb
Commit b82325cb authored Mar 05, 2018 by Robert Speicher
Hide 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
@@ -253,7 +253,7 @@ class Repository
  # branches or tags, but we want to keep some of these commits around, for
  # example if they have comments or CI builds.
  def keep_around(sha)
-    return unless sha && commit_by(oid: sha)
+    return unless sha.present? && commit_by(oid: sha)

    return if kept_around?(sha)