Skip to content

Commit d361308

Browse files
committed
Allow bot reviewers
1 parent 5724162 commit d361308

File tree

2 files changed

+3
-4
lines changed

2 files changed

+3
-4
lines changed

src/github/pullRequestModel.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1134,8 +1134,8 @@ export class PullRequestModel extends IssueModel<PullRequest> implements IPullRe
11341134
owner: remote.owner,
11351135
repo: remote.repositoryName,
11361136
pull_number: this.number,
1137-
reviewers: reviewers.filter(r => r.accountType !== AccountType.Bot).map(r => r.id),
1138-
team_reviewers: teamReviewers.map(t => t.id)
1137+
reviewers: reviewers.map(r => r.id),
1138+
team_reviewers: teamReviewers.map(t => t.id),
11391139
});
11401140

11411141
this.reviewers = this.reviewers?.filter(r => !reviewers.some(rr => rr.id === r.id) && !teamReviewers.some(t => t.id === r.id)) || [];

src/github/pullRequestOverview.ts

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -424,8 +424,7 @@ export class PullRequestOverviewPanel extends IssueOverviewPanel<PullRequestMode
424424
quickPick.busy = false;
425425
const acceptPromise: Promise<(IAccount | ITeam)[]> = asPromise<void>(quickPick.onDidAccept).then(() => {
426426
const pickedReviewers: (IAccount | ITeam)[] | undefined = quickPick?.selectedItems.filter(item => item.user).map(item => item.user) as (IAccount | ITeam)[];
427-
const botReviewers = this._existingReviewers.filter(reviewer => !isITeam(reviewer.reviewer) && reviewer.reviewer.accountType === 'Bot').map(reviewer => reviewer.reviewer);
428-
return pickedReviewers.concat(botReviewers);
427+
return pickedReviewers;
429428
});
430429
const hidePromise = asPromise<void>(quickPick.onDidHide);
431430
const allReviewers = await Promise.race<(IAccount | ITeam)[] | void>([acceptPromise, hidePromise]);

0 commit comments

Comments
 (0)