Fix dumb mistake

JelteF · JelteF · commit 5ba8175370f6 · 2025-09-04T10:53:38.000+02:00
We were ignoring all patches...
diff --git a/cfbot_commitfest_rpc.py b/cfbot_commitfest_rpc.py
@@ -44,7 +44,7 @@ def get_latest_patches_from_thread_url(thread_url):
             '<a href="(/message-id/attachment/[^"]*\\.(diff|diff\\.gz|patch|patch\\.gz|tar\\.gz|tgz|tar\\.bz2|zip))">',
             line,
         )
-        if groups and not groups.group(1):
+        if groups:
             attachment = groups.group(1)
             if "/nocfbot" not in attachment and not attachment.endswith(
                 "subtrans-benchmark.tar.gz"

Original file line number	Diff line number	Diff line change
`@@ -44,7 +44,7 @@ def get_latest_patches_from_thread_url(thread_url):`
`44`	`44`	`'<a href="(/message-id/attachment/[^"]*\\.(diff\|diff\\.gz\|patch\|patch\\.gz\|tar\\.gz\|tgz\|tar\\.bz2\|zip))">',`
`45`	`45`	`line,`
`46`	`46`	`)`
`47`		`- if groups and not groups.group(1):`
	`47`	`+ if groups:`
`48`	`48`	`attachment = groups.group(1)`
`49`	`49`	`if "/nocfbot" not in attachment and not attachment.endswith(`
`50`	`50`	`"subtrans-benchmark.tar.gz"`