ReviewOS

also looking at this

stacks/buddy

fix: give GitLab/Bitbucket pipelines the review job, and stop cleanup lying

#1432
Merged glennmichael123 wants to merge fix/pipelines-and-cleanup into main
5 files +152 -8
src/buddy.tsmodified+9-2
Changes to src/buddy.ts
@@ -752,8 +752,15 @@ export class Buddy {
752752 }
753753
754754 await gitProvider.closePullRequest(existingPR.number)
755 await gitProvider.deleteBranch(existingPR.head)
756 this.logger.success(`✅ Auto-closed PR #${existingPR.number} and deleted branch ${existingPR.head}`)
755 // The pull request is closed either way; a branch that would
756 // not delete is worth a warning, not a failed run.
757 try {
758 await gitProvider.deleteBranch(existingPR.head)
759 this.logger.success(`✅ Auto-closed PR #${existingPR.number} and deleted branch ${existingPR.head}`)
760 }
761 catch (deleteError) {
762 this.logger.warn(`⚠️ Auto-closed PR #${existingPR.number} but could not delete ${existingPR.head}:`, deleteError)
763 }
757764 continue
758765 }
759766 catch (error) {