mr discussions: Set CommitID

prarit · prarit · commit 31ae4a7466b4 · 2021-09-23T14:30:29.000-04:00
The CommitID now must be set when creating a merge request discussion with
commit comments.

Set the CommitID field when creating a merge request discussion with
commit comments.

Signed-off-by: Prarit Bhargava &lt;prarit@redhat.com&gt;
diff --git a/internal/gitlab/gitlab.go b/internal/gitlab/gitlab.go
@@ -1889,6 +1889,7 @@ func CreateMergeRequestCommitDiscussion(project string, mrID int, sha string, ne
 	opt := &gitlab.CreateMergeRequestDiscussionOptions{
 		Body:     &comment,
 		Position: &position,
+		CommitID: &sha,
 	}
 
 	discussion, _, err := lab.Discussions.CreateMergeRequestDiscussion(p.ID, mrID, opt)

Original file line number	Diff line number	Diff line change
`@@ -1889,6 +1889,7 @@ func CreateMergeRequestCommitDiscussion(project string, mrID int, sha string, ne`
`1889`	`1889`	`opt := &gitlab.CreateMergeRequestDiscussionOptions{`
`1890`	`1890`	`Body: &comment,`
`1891`	`1891`	`Position: &position,`
	`1892`	`+ CommitID: &sha,`
`1892`	`1893`	`}`
`1893`	`1894`
`1894`	`1895`	`discussion, _, err := lab.Discussions.CreateMergeRequestDiscussion(p.ID, mrID, opt)`