Same reviewer tagged several times.

[maksymiliev]

It seems like the bot has an issue where it fails to tag a correct reviewer and instead tags a different reviewer, resulting in one reviewer being tagged several times.

In the screenshot below you can see @maksym Iliev has been tagged 2 times, even though there should've been a different person tagged the second time instead of tagging @maksym Iliev again.

The Reviewer file has the correct mapping of gitlab username to slack ID just like provided in examples.

[maksymiliev]

Update:

Issue seems to be related with how the bot assesses reviewers that have approved the pull/merge request and those that haven't.

Instead, I have changed the template file to simply show all reviewers in the merge/pull request:

*<{{.Project.WebURL}}|{{.Project.Name}}>* Merge Requests:


{{range .Reminders}}
*<{{.MR.WebURL}}|{{.MR.Title}}>*
{{if .Discussions}} {{.Discussions}} :speech_balloon: {{end}} {{range $emoji, $count := .Emojis}} {{$count}} :{{$emoji}}: {{end}}
{{range .MR.Reviewers}}
<@{{.Username}}> {{end}}
{{end}}

[sj14]

Thanks for the feedback. This needs investigation.

[sj14]

review-bot/hoster/gitlab/gitlab.go

Line 162 in 8468066

if mr.Author != nil {

getReviewed() is only considering the Author of the MR as the possible owner, but the owner could be determined as someone else later in the processing.

review-bot/hoster/gitlab/gitlab.go

Line 93 in 8468066

func responsiblePerson(mr *gitlab.MergeRequest, reviewers map[string]string) string {

Maybe that's the issue but at the moment I don't have a working test setup.