Fix dlang#191 - remove common attributes when not in PR title

berni44 · Feb 21, 2021 · 6ecbd56 · 6ecbd56
1 parent 10ce8a1
commit 6ecbd56
Showing 1 changed file with 10 additions and 2 deletions.
diff --git a/source/dlangbot/github.d b/source/dlangbot/github.d
@@ -323,8 +323,8 @@ void searchForAutoMergePrs(string repoSlug)
 
 /**
 Allows contributors to use [<label>] messages in the title.
-If they are part of a pre-defined, allowed list, the bot will add the
-respective label.
+If they are part of a pre-defined, allowed list, the bot will add and
+remove the respective label.
 */
 void checkTitleForLabels(in ref PullRequest pr)
 {
@@ -356,4 +356,12 @@ void checkTitleForLabels(in ref PullRequest pr)
 
     if (mappedLabels.length)
         pr.addLabels(mappedLabels);
+
+    auto notMappedLabels = userLabelsMap
+                               .values
+                               .filter!(l => !mappedLabels.canFind(l))
+                               .array;
+
+    if (notMappedLabels.length)
+        checkAndRemoveLabels(pr.labels, pr, notMappedLabels);
 }