Merge pull request #45 from geeksesi/fix/43-update-pull-request

fix: Don't create a closed PR
CodeReviewPals · Apr 10, 2023 · 85001e2 · 85001e2
2 parents 28efdfb + 7d20b38
commit 85001e2
Showing 1 changed file with 5 additions and 0 deletions.
diff --git a/app/Services/Github/PullRequestService.php b/app/Services/Github/PullRequestService.php
@@ -4,6 +4,7 @@
 
 use Exception;
 use App\Models\PullRequest;
+use App\Enums\Github\Repository\PullRequest\State;
 use App\DTO\Github\Repository\PullRequest\PullRequestData;
 use App\Http\Requests\PullRequest\StorePullRequestRequest;
 use App\Actions\Github\PullRequest\FetchPullRequestInformation;
@@ -26,6 +27,10 @@ public function create(StorePullRequestRequest $request): ?PullRequest
             return null;
         }
 
+        if ($pullRequestData->state === State::CLOSED) {
+            return null;
+        }
+
         return $request
             ->user()
             ?->pullRequests()