Merge pull request #2342 from kpedro88/avoid_dup

avoid duplication in l2 list
cms-sw · Sep 24, 2024 · 357ff4b · 357ff4b
2 parents 31e877d + 60cf21a
commit 357ff4b
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 2 deletions.
diff --git a/process_pr.py b/process_pr.py
@@ -2189,6 +2189,7 @@ def process_pr(repo_config, gh, repo, issue, dryRun, cmsbuild_user=None, force=F
             for signature in new_assign_cats
             if signature in l2_categories
         ]
+        new_l2s = sorted(set(new_l2s), key=lambda s: s.lower())
         if not dryRun:
             issue.create_comment(
                 "New categories assigned: "

diff --git a/tests/PRActionData/TestProcessPr.test_assign.json b/tests/PRActionData/TestProcessPr.test_assign.json
@@ -80,7 +80,7 @@
     },
     {
         "type": "create-comment",
-        "data": "New categories assigned: dqm,db\n\nrvenditti,syuvivida,tjavaid,nothingface0,antoniovagnerini,francescobrivio,saumyaphor4252,perrotta,consuegs you have been requested to review this Pull request/Issue and eventually sign? Thanks"
+        "data": "New categories assigned: dqm,db\n\nantoniovagnerini,consuegs,francescobrivio,nothingface0,perrotta,rvenditti,saumyaphor4252,syuvivida,tjavaid you have been requested to review this Pull request/Issue and eventually sign? Thanks"
     },
     {
         "type": "add-label",
@@ -107,4 +107,4 @@
             "context": "bot/17/ack"
         }
     }
-]
+]