Skip to content

Restore User.from_gitaly

Jacob Vosmaer (GitLab) requested to merge restore-user-from-gitaly into master

What does this MR do?

Restores Gitlab::Git::User.from_gitaly which was accidentally removed in https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/13264.

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

Why was this MR needed?

The removal of this code breaks the vendoring of lib/gitlab/git into gitaly-ruby.

Screenshots (if relevant)

Does this MR meet the acceptance criteria?

What are the relevant issue numbers?

Broken Gitaly MR: https://gitlab.com/gitlab-org/gitaly/merge_requests/389

Issue to fix TODO introduced in this MR: https://gitlab.com/gitlab-org/gitaly/issues/628

Edited by Jacob Vosmaer (GitLab)

Merge request reports