commit 16d723ee1be9bcb07e44eefaeaa2d2e8786010ce
author Fabio Baltieri <fabiobaltieri@google.com> Thu Jan 26 17:03:13 2023 +0000
committer Anas Nashif <anas.nashif@intel.com> Thu Jan 26 14:57:29 2023 -0500
tree cb7429da2647bba3a7b32a105d5ef337ddd53074
parent ac5d45a080694f450e274b85f16441a8499ff391

scripts: set_assignee: don't skip assignment on too many labels

The set assignee script currently exits if there's too many labels
associated with a PR. Change that to just skip the label assignment, but
continue on to assign a maintainer based on the most relevant area.

Tested with:

  ./scripts/set_assignees.py -v --dry-run -P 52716

Signed-off-by: Fabio Baltieri <fabiobaltieri@google.com>

scripts/set_assignees.py

diff --git a/scripts/set_assignees.py b/scripts/set_assignees.py
index a636098..09571ab 100755
--- a/scripts/set_assignees.py
+++ b/scripts/set_assignees.py
@@ -80,9 +80,6 @@
     ac = dict(sorted(area_counter.items(), key=lambda item: item[1], reverse=True))
     log(f"Area matches: {ac}")
     log(f"labels: {labels}")
-    if len(labels) > 10:
-        log(f"Too many labels to be applied")
-        return
 
     # Create a list of collaborators ordered by the area match
     collab = list()
@@ -143,11 +140,14 @@
     log("+++++++++++++++++++++++++")
 
     # Set labels
-    if labels and len(labels) < 10:
-        for l in labels:
-            log(f"adding label {l}...")
-            if not args.dry_run:
-                pr.add_to_labels(l)
+    if labels:
+        if len(labels) < 10:
+            for l in labels:
+                log(f"adding label {l}...")
+                if not args.dry_run:
+                    pr.add_to_labels(l)
+        else:
+            log(f"Too many labels to be applied")
 
     if collab:
         reviewers = []