Skip to content

Commit af2c668

Browse files
fix: add team_reviewers to requestReviewers (#276)
* fix: add team_reviewers to requestReviewers * chore: move comment * chore: update team reviewer to just slug * fix: add backup case * refactor: update slug for readability
1 parent c0af5d0 commit af2c668

File tree

1 file changed

+9
-3
lines changed

1 file changed

+9
-3
lines changed

src/utils.ts

+9-3
Original file line numberDiff line numberDiff line change
@@ -407,23 +407,29 @@ export const tagBackportReviewers = async ({
407407
user?: string;
408408
}) => {
409409
const reviewers = [];
410+
const teamReviewers = [];
410411

411412
if (DEFAULT_BACKPORT_REVIEW_TEAM) {
412-
reviewers.push(DEFAULT_BACKPORT_REVIEW_TEAM);
413+
// Optionally request a default review team for backports.
414+
// Use team slug value. i.e electron/wg-releases => wg-releases
415+
const slug =
416+
DEFAULT_BACKPORT_REVIEW_TEAM.split('/')[1] ||
417+
DEFAULT_BACKPORT_REVIEW_TEAM;
418+
teamReviewers.push(slug);
413419
}
414420

415421
if (user) {
416422
const hasWrite = await checkUserHasWriteAccess(context, user);
417-
// Optionally request a default review team for backports.
418423
// If the PR author has write access, also request their review.
419424
if (hasWrite) reviewers.push(user);
420425
}
421426

422-
if (reviewers.length > 0) {
427+
if (Math.max(reviewers.length, teamReviewers.length) > 0) {
423428
await context.octokit.pulls.requestReviewers(
424429
context.repo({
425430
pull_number: targetPrNumber,
426431
reviewers,
432+
team_reviewers: teamReviewers,
427433
}),
428434
);
429435
}

0 commit comments

Comments
 (0)