Skip to content

Commit 2b3fa79

Browse files
committed
bug #156 [Reviewer] Verify that the issue is still open (Nyholm)
This PR was merged into the master branch. Discussion ---------- [Reviewer] Verify that the issue is still open Before we suggest a reviewer, make sure the issue is still open. It might be the case where an issue is merged without any activity from other users than the author. FYI @OskarStark Commits ------- 7c8568b Verify that the issue is still open
2 parents 3253891 + 7c8568b commit 2b3fa79

File tree

1 file changed

+5
-0
lines changed

1 file changed

+5
-0
lines changed

src/Service/TaskHandler/SuggestReviewerHandler.php

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -48,6 +48,11 @@ public function handle(Task $task): void
4848
return;
4949
}
5050

51+
$issue = $this->issueApi->show($repository, $task->getNumber());
52+
if ('open' !== $issue['state']) {
53+
return;
54+
}
55+
5156
if ($this->issueApi->hasActivity($repository, $task->getNumber())) {
5257
return;
5358
}

0 commit comments

Comments
 (0)