From 03b2715858d4816392e647978c743c538dc652a1 Mon Sep 17 00:00:00 2001 From: Mattia Verga Date: Oct 04 2023 11:38:06 +0000 Subject: Better explain pagure comments Signed-off-by: Mattia Verga --- diff --git a/find_inactive_packagers.py b/find_inactive_packagers.py index 8fbc9fc..4c42459 100644 --- a/find_inactive_packagers.py +++ b/find_inactive_packagers.py @@ -109,8 +109,9 @@ if your email set in Fedora Account System ({email}) is still valid. Please, let us know if you\'re still intersted in participating in Fedora and if you still need your account to be listed in the `packager` group. -Without any reply from you, in two months we will proceed to remove -your account from the `packager` group. Your account will still be active. +Without any reply from you, in two months the `packager` group membership will be +removed from your account and any package for which you are the main admin will +be orphaned, so that co-maintainers can pick them up. ''' CHECK_IMPACT_OUTPUT = '''### Found {total_packages} packages which will possibly be orphaned: ### @@ -126,11 +127,16 @@ Have a wonderful day and see you (maybe?) at the next run! ''' CHECK_IMPACT_COMMENT = '''The following is a list of packages that may become orphan if the user -is removed from the `packager` group. Affected co-maintainers users and groups are provided -as well: +@{username} is removed from the `packager` group. Affected co-maintainers users and groups are +provided as well: {package_list} +Note that this is just an early warning for package co-maintainers/collaborators: the maintainer +can still show up before the orphaning happen. +Anyone who wants to become the maintainer of a package before it is orphaned can try to contact +the current maintainer themselves or, if you **really** can't wait after the automatic orphaning, +open a ticket to Fedora Releng. ''' logging.basicConfig(level=logging.INFO, @@ -890,7 +896,7 @@ def check_impact(ctx, comment, otf): user_package_map.append((package, affected_users, affected_groups)) package_list_comment = "\n".join([f'- {package}{" (affected users: "+affected_users+")" if affected_users else ""}{" (affected groups: "+affected_groups+")" if affected_groups else ""}' for package, affected_users, affected_groups in user_package_map]) if package_list_comment: - _comment_pagure_ticket(t, CHECK_IMPACT_COMMENT.format(package_list=package_list_comment)) + _comment_pagure_ticket(t, CHECK_IMPACT_COMMENT.format(package_list=package_list_comment, username=u)) @cli.command()