From 1e2341e1a349b021fec61606075c43055c7673d6 Mon Sep 17 00:00:00 2001 From: Shahzad Malik Muzaffar Date: Tue, 26 Nov 2024 23:44:25 +0100 Subject: [PATCH] only auto sign operations if there are other cats too --- process_pr.py | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/process_pr.py b/process_pr.py index 8cfb23af2bf..6c7a9345286 100644 --- a/process_pr.py +++ b/process_pr.py @@ -2085,7 +2085,8 @@ def process_pr(repo_config, gh, repo, issue, dryRun, cmsbuild_user=None, force=F labels.append("tests-" + override_tests_failure) # automatically approve CATS_TO_APPROVE_ON_TEST on test-approved - if signatures.get("tests") == "approved": + xcats = CATS_TO_APPROVE_ON_TEST + ["orp", "tests", "code-checks"] + if (signatures.get("tests") == "approved") and [c for c in signatures if not c in xcats]: for cat in [c for c in CATS_TO_APPROVE_ON_TEST if (signatures.get(c) == "pending")]: signatures[cat] = "approved" print("Overriding/Approving singatures for %s due to tests-approved" % cat)