From e88e7fe4cda08ef1a5dcdc01494b9ce18b695014 Mon Sep 17 00:00:00 2001 From: Spencer Murray Date: Thu, 13 Mar 2025 18:21:21 -0400 Subject: [PATCH 1/2] Allow PRs from sentry-autofix[bot] without needing seat --- services/decoration.py | 3 ++- services/tests/test_decoration.py | 2 ++ 2 files changed, 4 insertions(+), 1 deletion(-) diff --git a/services/decoration.py b/services/decoration.py index 9fe6bc0d6..4aea8b79f 100644 --- a/services/decoration.py +++ b/services/decoration.py @@ -19,8 +19,9 @@ BOT_USER_EMAILS = [ "dependabot[bot]@users.noreply.github.com", "29139614+renovate[bot]@users.noreply.github.com", + "157164994+sentry-autofix[bot]@users.noreply.github.com", ] -BOT_USER_IDS = ["29139614"] # renovate[bot] github +BOT_USER_IDS = ["29139614", "157164994"] # renovate[bot] github, sentry-autofix[bot] USER_BASIC_LIMIT_UPLOAD = 250 diff --git a/services/tests/test_decoration.py b/services/tests/test_decoration.py index 8ff0713d7..cfced3a2e 100644 --- a/services/tests/test_decoration.py +++ b/services/tests/test_decoration.py @@ -612,7 +612,9 @@ def test_get_decoration_type_pr_author_manual_activation_required( [ (True, "email", "dependabot[bot]@users.noreply.github.com"), (True, "email", "29139614+renovate[bot]@users.noreply.github.com"), + (True, "email", "157164994+sentry-autofix[bot]@users.noreply.github.com"), (True, "service_id", "29139614"), + (True, "service_id", "157164994"), (False, None, None), ], ) From d675f57a0baa2096536d8824f96150f5334f0fca Mon Sep 17 00:00:00 2001 From: Spencer Murray Date: Thu, 13 Mar 2025 18:27:55 -0400 Subject: [PATCH 2/2] Run lint idk --- tasks/new_user_activated.py | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/tasks/new_user_activated.py b/tasks/new_user_activated.py index 03b0f6f63..02eb427d6 100644 --- a/tasks/new_user_activated.py +++ b/tasks/new_user_activated.py @@ -70,10 +70,10 @@ def run_impl(self, db_session, org_ownerid, user_ownerid, *args, **kwargs): ) return { - "notifies_scheduled": bool(len(pulls_notified)), + "notifies_scheduled": bool(pulls_notified), "pulls_notified": pulls_notified, "reason": None - if len(pulls_notified) + if pulls_notified else "no pulls/pull notifications met criteria", }