proj/assign-pull-requests:master commit in: /
"Thomas Bracht Laumann Jespersen" <[email protected]>
| Newsgroups | gmane.linux.gentoo.cvs |
|---|---|
| Message-ID | <1786078852.1d06fa1b8597abbc31811d30a2b90c712c377b24.laumann@gentoo> |
commit: 1d06fa1b8597abbc31811d30a2b90c712c377b24
Author: Thomas Bracht Laumann Jespersen <laumann <AT> gentoo <DOT> org>
AuthorDate: Fri Aug 7 05:00:52 2026 +0000
Commit: Thomas Bracht Laumann Jespersen <laumann <AT> gentoo <DOT> org>
CommitDate: Fri Aug 7 05:00:52 2026 +0000
URL: https://gitweb.gentoo.org/proj/assign-pull-requests.git/commit/?id=1d06fa1b
codeberg: fix incorrect "New Packages" notifications
In PRs that touch existing packages without maintainers, we should not
flag them as adding new packages. The logic should be: If an existing
metadata.xml is found, then it's an existing package. This mirrors
what the github PR assignment does (in other words, yes, I broke it).
Signed-off-by: Thomas Bracht Laumann Jespersen <laumann <AT> gentoo.org>
assign-pull-requests-codeberg.py | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/assign-pull-requests-codeberg.py b/assign-pull-requests-codeberg.py
index 2677f73..81d0e0f 100644
--- a/assign-pull-requests-codeberg.py
+++ b/assign-pull-requests-codeberg.py
@@ -271,9 +271,10 @@ def assign_one(
all_ms = ["@gentoo/proxy-maint (new package)"]
team_reviewers.add("proxy-maint")
new_package = True
+ else:
+ existing_package = True
for memail in emails:
- existing_package = True
totally_all_maints.add(memail)
ms, reviewer = map_email(memail.lower(), dev_mapping, proj_mapping)
if reviewer: