Fix #1555: Avoid duplication of pull reqyuest bypasser (#1711)

bpaquet · kfcampbell · web-flow · commit be7cfe80e399 · 2023-06-22T13:30:36.000-07:00
Co-authored-by: Keegan Campbell &lt;me@kfcampbell.com&gt;
diff --git a/github/util_v4_branch_protection.go b/github/util_v4_branch_protection.go
@@ -411,6 +411,7 @@ func setBypassPullRequestActorIDs(actors []BypassPullRequestActorTypes, data Bra
 			}
 			if a.Actor.User.Login != "" {
 				bypassActors = append(bypassActors, "/"+string(a.Actor.User.Login))
+				continue
 			}
 			if a.Actor.App != (Actor{}) {
 				bypassActors = append(bypassActors, a.Actor.App.ID.(string))

Original file line number	Diff line number	Diff line change
`@@ -411,6 +411,7 @@ func setBypassPullRequestActorIDs(actors []BypassPullRequestActorTypes, data Bra`
`411`	`411`	`}`
`412`	`412`	`if a.Actor.User.Login != "" {`
`413`	`413`	`bypassActors = append(bypassActors, "/"+string(a.Actor.User.Login))`
	`414`	`+ continue`
`414`	`415`	`}`
`415`	`416`	`if a.Actor.App != (Actor{}) {`
`416`	`417`	`bypassActors = append(bypassActors, a.Actor.App.ID.(string))`