Skip to content

Commit 4b5471d

Browse files
authored
Merge pull request #3382 from wojtek-t/cleanup_prr_approvers
Cleanup PRR approvers fetching
2 parents 9e1a8ed + 8c2e0d3 commit 4b5471d

File tree

1 file changed

+0
-4
lines changed

1 file changed

+0
-4
lines changed

api/groups.go

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -161,10 +161,6 @@ func (f *RemoteGroupFetcher) FetchPRRApprovers() ([]string, error) {
161161
result = append(result, config.Data[f.PRRApproversAlias]...)
162162
// TODO: Figre out if we want to treat emeritus approvers differently.
163163
result = append(result, config.Data[f.PRRApproversEmeritusAlias]...)
164-
// TODO(wojtekt): This is temporarily hack to ensure we can merge the PR,
165-
// because our testing is fetching approvers from head.
166-
// Get rid of it as soon as it merges :)
167-
result = append(result, "ehashman")
168164

169165
if len(result) == 0 {
170166
return nil, errors.New("retrieved zero PRR approvers, which is unexpected")

0 commit comments

Comments
 (0)