diff options
-rw-r--r-- | app/models/commit.rb | 2 | ||||
-rw-r--r-- | lib/gitlab/diff/diff_refs.rb | 4 |
2 files changed, 2 insertions, 4 deletions
diff --git a/app/models/commit.rb b/app/models/commit.rb index c85b5b93607..6dba154a6ea 100644 --- a/app/models/commit.rb +++ b/app/models/commit.rb @@ -25,7 +25,6 @@ class Commit DIFF_HARD_LIMIT_FILES = 1000 DIFF_HARD_LIMIT_LINES = 50000 - # The SHA can be between 7 and 40 hex characters. MIN_SHA_LENGTH = 7 COMMIT_SHA_PATTERN = /\h{#{MIN_SHA_LENGTH},40}/.freeze @@ -217,7 +216,6 @@ class Commit @raw.respond_to?(method, include_private) || super end - # Truncate sha to 7 characters def short_id @raw.short_id(MIN_SHA_LENGTH) end diff --git a/lib/gitlab/diff/diff_refs.rb b/lib/gitlab/diff/diff_refs.rb index 1c4c1deb7a9..c98eefbce25 100644 --- a/lib/gitlab/diff/diff_refs.rb +++ b/lib/gitlab/diff/diff_refs.rb @@ -57,8 +57,8 @@ module Gitlab length = [sha1.length, sha2.length].min - # If either of the shas is below 7 characters in length, we cannot be sure - # if they actually refer to the same commit because of hash collision. + # If either of the shas is below the minimum length, we cannot be sure + # that they actually refer to the same commit because of hash collision. return false if length < Commit::MIN_SHA_LENGTH sha1[0, length] == sha2[0, length] |