From a3a89bb346763b7f7e0db99d667f0d19b0aa91c1 Mon Sep 17 00:00:00 2001
From: Karma Riuk <riukkarma@gmail.com>
Date: Tue, 20 May 2025 09:58:49 +0200
Subject: [PATCH] moved some logic around

---
 pull_requests.py | 8 +++++++-
 1 file changed, 7 insertions(+), 1 deletion(-)

diff --git a/pull_requests.py b/pull_requests.py
index 81b91a2..36c2143 100644
--- a/pull_requests.py
+++ b/pull_requests.py
@@ -41,6 +41,9 @@ EXCLUSION_LIST = [
 
 
 def is_pull_good(pull: PullRequest, verbose: bool = False) -> bool:
+    if pull.merged_at is None:
+        return False
+
     comments = pull.get_review_comments()
     if pull.user.type == "Bot" or comments.totalCount > 2 or comments.totalCount == 0:
         return False
@@ -229,6 +232,9 @@ def process_pull(
         dataset.entries.append(cache[repo.full_name][pr.number])
         return
 
+    if not is_pull_good(pr):
+        return
+
     metadata = Metadata(
         uuid.uuid4().hex,
         repo.full_name,
@@ -407,7 +413,7 @@ def process_repo(
         for pr in prs:
             pbar.set_postfix({"pr": pr.number, "# new good found": n_good_prs})
             try:
-                if pr.merged_at is None or pr.number in already_seen_prs or not is_pull_good(pr):
+                if pr.number in already_seen_prs:
                     continue
 
                 n_good_prs += 1