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:
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.