removed request for code owners (#6828)

Fixes #6788

Author: alexr00

src/github/activityBarViewProvider.ts

@@ -157,23 +157,21 @@ export class PullRequestViewProvider extends WebviewViewBase implements vscode.W
 		const teamReviewers: ITeam[] = [];
 
 		for (const reviewer of this._existingReviewers) {
-			let id: string | undefined;
-			let reviewerArray: (IAccount | ITeam)[] | undefined;
-			if (reviewer && isTeam(reviewer.reviewer)) {
-				id = reviewer.reviewer.id;
-				reviewerArray = teamReviewers;
-			} else if (reviewer && !isTeam(reviewer.reviewer)) {
-				id = reviewer.reviewer.id;
-				reviewerArray = userReviewers;
-			}
-			if (reviewerArray && id && ((reviewer.state === 'REQUESTED') || (id === message.args))) {
-				reviewerArray.push(reviewer.reviewer);
+			let id = reviewer.reviewer.id;
+			if (id && ((reviewer.state === 'REQUESTED') || (id === message.args))) {
 				if (id === message.args) {
 					targetReviewer = reviewer;
 				}
 			}
 		}
-		this._item.requestReview(userReviewers, teamReviewers).then(() => {
+
+		if (targetReviewer && isTeam(targetReviewer.reviewer)) {
+			teamReviewers.push(targetReviewer.reviewer);
+		} else if (targetReviewer && !isTeam(targetReviewer.reviewer)) {
+			userReviewers.push(targetReviewer.reviewer);
+		}
+
+		this._item.requestReview(userReviewers, teamReviewers, true).then(() => {
 			if (targetReviewer) {
 				targetReviewer.state = 'REQUESTED';
 			}

src/github/createPRViewProvider.ts

@@ -316,7 +316,7 @@ export abstract class BaseCreatePullRequestViewProvider<T extends BasePullReques
 					users.push(reviewer);
 				}
 			}
-			await pr.requestReview(users, teams);
+			await pr.requestReview(users, teams, true);
 		}
 	}
 

src/github/pullRequestModel.ts

@@ -962,7 +962,7 @@ export class PullRequestModel extends IssueModel<PullRequest> implements IPullRe
 	 * Add reviewers to a pull request
 	 * @param reviewers A list of GitHub logins
 	 */
-	async requestReview(reviewers: IAccount[], teamReviewers: ITeam[]): Promise<void> {
+	async requestReview(reviewers: IAccount[], teamReviewers: ITeam[], union: boolean = false): Promise<void> {
 		const { mutate, schema } = await this.githubRepository.ensure();
 		await mutate({
 			mutation: schema.AddReviewers,
@@ -971,6 +971,7 @@ export class PullRequestModel extends IssueModel<PullRequest> implements IPullRe
 					pullRequestId: this.graphNodeId,
 					teamIds: teamReviewers.map(t => t.id),
 					userIds: reviewers.filter(r => r.accountType !== AccountType.Bot).map(r => r.id),
+					union
 				},
 			},
 		});

src/github/pullRequestOverview.ts

@@ -653,24 +653,21 @@ export class PullRequestOverviewPanel extends IssueOverviewPanel<PullRequestMode
 		const teamReviewers: ITeam[] = [];
 
 		for (const reviewer of this._existingReviewers) {
-			let id: string | undefined;
-			let reviewerArray: (IAccount | ITeam)[] | undefined;
-			if (reviewer && isTeam(reviewer.reviewer)) {
-				id = reviewer.reviewer.id;
-				reviewerArray = teamReviewers;
-			} else if (reviewer && !isTeam(reviewer.reviewer)) {
-				id = reviewer.reviewer.id;
-				reviewerArray = userReviewers;
-			}
-			if (reviewerArray && id && ((reviewer.state === 'REQUESTED') || (id === message.args))) {
-				reviewerArray.push(reviewer.reviewer);
+			let id = reviewer.reviewer.id;
+			if (id && ((reviewer.state === 'REQUESTED') || (id === message.args))) {
 				if (id === message.args) {
 					targetReviewer = reviewer;
 				}
 			}
 		}
 
-		this._item.requestReview(userReviewers, teamReviewers).then(() => {
+		if (targetReviewer && isTeam(targetReviewer.reviewer)) {
+			teamReviewers.push(targetReviewer.reviewer);
+		} else if (targetReviewer && !isTeam(targetReviewer.reviewer)) {
+			userReviewers.push(targetReviewer.reviewer);
+		}
+
+		this._item.requestReview(userReviewers, teamReviewers, true).then(() => {
 			if (targetReviewer) {
 				targetReviewer.state = 'REQUESTED';
 			}