delete should happen regardless if there are tickets associated (#1240)

crs-k · Aug 5, 2024 · 73d715e · 73d715e
1 parent b62e20c
commit 73d715e
Show file tree

Hide file tree

Showing 3 changed files with 14 additions and 14 deletions.
diff --git a/dist/index.js b/dist/index.js
diff --git a/dist/index.js.map b/dist/index.js.map
diff --git a/src/stale-branches.ts b/src/stale-branches.ts
@@ -154,15 +154,15 @@ export async function run(): Promise<void> {
 
       //Delete expired branches
       if (commitAge > validInputs.daysBeforeDelete && branchComparison.save === false) {
+        if (!validInputs.dryRun) {
+          await deleteBranch(branchToCheck.branchName)
+          outputDeletes.push(branchToCheck.branchName)
+        } else {
+          core.info(`Dry Run: Branch would be deleted: ${branchToCheck.branchName}`)
+        }
         for (const issueToDelete of filteredIssue) {
           if (issueToDelete.issueTitle === issueTitleString) {
-            if (!validInputs.dryRun) {
-              await deleteBranch(branchToCheck.branchName)
-            } else {
-              core.info(`Dry Run: Branch would be deleted: ${branchToCheck.branchName}`)
-            }
             closeIssueWrappedLogs(issueToDelete.issueNumber, validInputs, branchToCheck.branchName)
-            outputDeletes.push(branchToCheck.branchName)
           }
         }
       }