Only remove assignees if issue doesn't have a needs or TPI label (microsoft#19659)

Kartik Raj · web-flow · commit 822889bd5dd6 · 2022-08-09T10:20:24.000-07:00
diff --git a/.github/workflows/issue-labels.yml b/.github/workflows/issue-labels.yml
@@ -39,6 +39,20 @@ jobs:
                 issue_number: context.issue.number,
                 labels: ['triage-needed']
               })
+              const currentAssignees = await github.rest.issues
+                .get({
+                    owner: context.repo.owner,
+                    repo: context.repo.repo,
+                    issue_number: context.issue.number,
+                })
+                .then((result) => result.data.assignees.map((a) => a.login));
+              const assigneesToRemove = currentAssignees.filter(a => !knownTriagers.includes(a));
+              github.rest.issues.removeAssignees({
+                owner: context.repo.owner,
+                repo: context.repo.repo,
+                issue_number: context.issue.number,
+                assignees: assigneesToRemove,
+              });
             } else {
               console.log('This issue already has a "needs __", "iteration-plan", "release-plan", or the "testplan-item" label, do not add the "triage-needed" label.')
             }
@@ -53,17 +67,3 @@ jobs:
                 })
               }
             }
-            const currentAssignees = await github.rest.issues
-              .get({
-                owner: context.repo.owner,
-                repo: context.repo.repo,
-                issue_number: context.issue.number,
-              })
-              .then((result) => result.data.assignees.map((a) => a.login));
-            const assigneesToRemove = currentAssignees.filter(a => !knownTriagers.includes(a));
-            github.rest.issues.removeAssignees({
-              owner: context.repo.owner,
-              repo: context.repo.repo,
-              issue_number: context.issue.number,
-              assignees: assigneesToRemove,
-            });
