From 450dfdb0c0c544d9cdd253a568555f2e8fd7d18b Mon Sep 17 00:00:00 2001
From: Anish Lakhwara <anish+git@lakhwara.com>
Date: Fri, 15 Nov 2024 12:05:24 -0800
Subject: [PATCH] feat: make team reviews a parameter

Previously, all PRs automatically added the sourcegrapm/team/release
reviewer to PRs. That worked fine while we were only using this on
Monorepo, but now we're expanding usage to Cody, Jetbrains and the
Deploy-* repos. Cody and Jetbrains particularly do not have the release
team as collaborator which caused errors while using this automation.

We're keeping the default the same with this change, but now we can
optionally override the default to change the team, or remove it
entirely (setting teamReviews = "").
---
 action.yml      | 4 ++++
 src/backport.ts | 7 ++++++-
 src/index.ts    | 2 ++
 3 files changed, 12 insertions(+), 1 deletion(-)

diff --git a/action.yml b/action.yml
index 4f3eff2..e9fa0b8 100644
--- a/action.yml
+++ b/action.yml
@@ -24,6 +24,10 @@ inputs:
         - base: backport PR's base branch
         - number: original PR's number
     default: "backport-<%= number %>-to-<%= base %>"
+  team_reviews:
+    description: Whether to request reviews from the team.
+    required: false
+    default: "release"
   label_pattern:
     description: >
       The regular expression pattern that PR labels will be tested on to decide whether the PR should be backported and where.
diff --git a/src/backport.ts b/src/backport.ts
index 983d8ed..ebbda97 100644
--- a/src/backport.ts
+++ b/src/backport.ts
@@ -87,6 +87,7 @@ const backportOnce = async ({
   merged_by,
   owner,
   repo,
+  teamReviews,
   title,
 }: Readonly<{
   author: string;
@@ -99,6 +100,7 @@ const backportOnce = async ({
   merged_by: string;
   owner: string;
   repo: string;
+  teamReviews: string;
   title: string;
 }>): Promise<number> => {
   const git = async (...args: string[]) => {
@@ -135,7 +137,7 @@ const backportOnce = async ({
         author !== merged_by && merged_by !== ""
           ? [author, merged_by]
           : [author],
-      team_reviewers: ["release"],
+      ...(teamReviews !== "" && { team_reviewers: [teamReviews] })
     },
   );
   if (labels.length > 0) {
@@ -243,6 +245,7 @@ const backport = async ({
   payload,
   runId,
   serverUrl,
+  teamReviews,
   token,
 }: {
   getBody: (
@@ -270,6 +273,7 @@ const backport = async ({
   payload: PullRequestClosedEvent | PullRequestLabeledEvent;
   runId: number;
   serverUrl: string;
+  teamReviews: string,
   token: string;
 }): Promise<{ [base: string]: number }> => {
   const {
@@ -354,6 +358,7 @@ const backport = async ({
           merged_by,
           owner,
           repo,
+          teamReviews,
           title,
         });
         createdPullRequestBaseBranchToNumber[base] = backportPullRequestNumber;
diff --git a/src/index.ts b/src/index.ts
index 6c27614..a2e3090 100644
--- a/src/index.ts
+++ b/src/index.ts
@@ -15,6 +15,7 @@ const run = async () => {
 
     const labelPattern = getInput("label_pattern");
     const labelRegExp = new RegExp(labelPattern);
+    const teamReviews = getInput("team_reviews");
 
     const token = getInput("github_token", { required: true });
 
@@ -40,6 +41,7 @@ const run = async () => {
       payload,
       runId,
       serverUrl,
+      teamReviews,
       token,
     });
     setOutput(