Really should use ||= assignment

16176fcb · Kerri Miller · 2671dd85 · 16176fcb
Commit 16176fcb authored Apr 06, 2020 by Kerri Miller
Hide whitespace changes
Inline Side-by-side

Showing with 1 addition and 1 deletion

ee/lib/gitlab/code_owners.rb ee/lib/gitlab/code_owners.rb +1 -1

No files found.
--- a/ee/lib/gitlab/code_owners.rb
+++ b/ee/lib/gitlab/code_owners.rb
@@ -56,7 +56,7 @@ module Gitlab
    end

    def self.slow_path_lookup(merge_request, merge_request_diff)
-      merge_request_diff = merge_request_diff || merge_request.merge_request_diff
+      merge_request_diff ||= merge_request.merge_request_diff

      merge_request.project.repository.diff_stats(
        merge_request_diff.base_commit_sha,