Skip to content

Refactor Gitlab::Git::Commit to include a repository and add missing specs

Alejandro Rodríguez requested to merge gitaly-9-5-2-patch into 9-5-stable-patch-3

What does this MR do?

Closes https://gitlab.com/gitlab-org/gitlab-ce/issues/37000

Refactor Gitlab::Git::Commit to include a repository and add missing specs

Are there points in the code the reviewer needs to double check?

No

Why was this MR needed?

We were getting production errors when Gitaly commits were being asked for their rugged commit counterpart, because of an incomplete migration of the changes included in https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/13094

Does this MR meet the acceptance criteria?

What are the relevant issue numbers?

Closes https://gitlab.com/gitlab-org/gitlab-ce/issues/37000

Edited by Andrew Newdigate

Merge request reports