From bf399e9bfb3b63e6e1a13b8a2624edcb4fb4457b Mon Sep 17 00:00:00 2001 From: Ryo Kitagawa Date: Thu, 11 Apr 2024 09:48:39 +0900 Subject: [PATCH] refactor: delete unused method --- pkg/notifier/gitlab/gen/gitlab.go | 21 --------------------- pkg/notifier/gitlab/gitlab.go | 6 ------ 2 files changed, 27 deletions(-) diff --git a/pkg/notifier/gitlab/gen/gitlab.go b/pkg/notifier/gitlab/gen/gitlab.go index 7f44257..d15383e 100644 --- a/pkg/notifier/gitlab/gen/gitlab.go +++ b/pkg/notifier/gitlab/gen/gitlab.go @@ -138,27 +138,6 @@ func (mr *MockAPIMockRecorder) GetMergeRequest(mergeRequest, opt any, options .. return mr.mock.ctrl.RecordCallWithMethodType(mr.mock, "GetMergeRequest", reflect.TypeOf((*MockAPI)(nil).GetMergeRequest), varargs...) } -// ListCommits mocks base method. -func (m *MockAPI) ListCommits(opt *gitlab.ListCommitsOptions, options ...gitlab.RequestOptionFunc) ([]*gitlab.Commit, *gitlab.Response, error) { - m.ctrl.T.Helper() - varargs := []any{opt} - for _, a := range options { - varargs = append(varargs, a) - } - ret := m.ctrl.Call(m, "ListCommits", varargs...) - ret0, _ := ret[0].([]*gitlab.Commit) - ret1, _ := ret[1].(*gitlab.Response) - ret2, _ := ret[2].(error) - return ret0, ret1, ret2 -} - -// ListCommits indicates an expected call of ListCommits. -func (mr *MockAPIMockRecorder) ListCommits(opt any, options ...any) *gomock.Call { - mr.mock.ctrl.T.Helper() - varargs := append([]any{opt}, options...) - return mr.mock.ctrl.RecordCallWithMethodType(mr.mock, "ListCommits", reflect.TypeOf((*MockAPI)(nil).ListCommits), varargs...) -} - // ListMergeRequestLabels mocks base method. func (m *MockAPI) ListMergeRequestLabels(mergeRequest int, opt *gitlab.GetMergeRequestsOptions, options ...gitlab.RequestOptionFunc) (gitlab.Labels, error) { m.ctrl.T.Helper() diff --git a/pkg/notifier/gitlab/gitlab.go b/pkg/notifier/gitlab/gitlab.go index 19cc00e..2ef96b4 100644 --- a/pkg/notifier/gitlab/gitlab.go +++ b/pkg/notifier/gitlab/gitlab.go @@ -14,7 +14,6 @@ type API interface { GetMergeRequest(mergeRequest int, opt *gitlab.GetMergeRequestsOptions, options ...gitlab.RequestOptionFunc) (*gitlab.MergeRequest, *gitlab.Response, error) UpdateMergeRequest(mergeRequest int, opt *gitlab.UpdateMergeRequestOptions, options ...gitlab.RequestOptionFunc) (*gitlab.MergeRequest, *gitlab.Response, error) PostCommitComment(sha string, opt *gitlab.PostCommitCommentOptions, options ...gitlab.RequestOptionFunc) (*gitlab.CommitComment, *gitlab.Response, error) - ListCommits(opt *gitlab.ListCommitsOptions, options ...gitlab.RequestOptionFunc) ([]*gitlab.Commit, *gitlab.Response, error) AddMergeRequestLabels(labels *[]string, mergeRequest int) (gitlab.Labels, error) RemoveMergeRequestLabels(labels *[]string, mergeRequest int) (gitlab.Labels, error) ListMergeRequestLabels(mergeRequest int, opt *gitlab.GetMergeRequestsOptions, options ...gitlab.RequestOptionFunc) (gitlab.Labels, error) @@ -60,11 +59,6 @@ func (g *GitLab) PostCommitComment(sha string, opt *gitlab.PostCommitCommentOpti return g.Client.Commits.PostCommitComment(fmt.Sprintf("%s/%s", g.namespace, g.project), sha, opt, options...) } -// ListCommits is a wrapper of https://godoc.org/github.com/xanzy/go-gitlab#CommitsService.ListCommits -func (g *GitLab) ListCommits(opt *gitlab.ListCommitsOptions, options ...gitlab.RequestOptionFunc) ([]*gitlab.Commit, *gitlab.Response, error) { - return g.Client.Commits.ListCommits(fmt.Sprintf("%s/%s", g.namespace, g.project), opt, options...) -} - // AddMergeRequestLabels adds labels on the merge request. func (g *GitLab) AddMergeRequestLabels(labels *[]string, mergeRequest int) (gitlab.Labels, error) { var addLabels gitlab.Labels