chore: add support for ignoring PRs by labels

Add `does_not_have_labels` parameter (list of strings) which will make
PR ignore PRs with those labels.

Author: adidenko

fixtures/config.yaml

@@ -28,6 +28,9 @@ github_pr_notifications:
     gh_pr_ignore_changes_requested: true
     gh_pr_conditions:
       older_than_seconds: 3600
+      does_not_have_labels:
+        - "WIP"
+        - "do not merge"
     # Mon-Fri every 2 hours during business hours
     schedule: "CRON_TZ=Europe/Berlin 00 10-18/2 * * 1-5"
     notify:

internal/cfg/cfg.go

@@ -79,7 +79,8 @@ type Notification struct {
 
 // PrConditions struct describes additional conditions for PRs
 type PrConditions struct {
-	OlderThanSeconds int `yaml:"older_than_seconds"`
+	OlderThanSeconds  int      `yaml:"older_than_seconds"`
+	DoesNotHaveLabels []string `yaml:"does_not_have_labels"`
 }
 
 // PrNotification is a struct for a single GH repo PRs notifications

internal/cfg/cfg_test.go

@@ -85,3 +85,31 @@ func TestGetSlackUID(t *testing.T) {
 		t.Errorf("Did not expect to find Slack user ID for GitHub login 'nonexistent-gh-user', but found one")
 	}
 }
+
+func TestPrConditions(t *testing.T) {
+	config := AppConfig{}
+	err := config.LoadConfig("../../fixtures/config.yaml")
+	if err != nil {
+		t.Errorf("Failed to load ./fixtures/config.yaml: %s", err.Error())
+	}
+
+	if len(config.PrNotifications) < 1 {
+		t.Errorf("Length of github_pr_notifications is %d, but it should be not empty", len(config.PrNotifications))
+	}
+
+	conditions := config.PrNotifications[0].Conditions
+	if conditions.OlderThanSeconds != 3600 {
+		t.Errorf("Expected gh_pr_conditions.older_than_seconds=3600, but got %d", conditions.OlderThanSeconds)
+	}
+
+	expectedLabels := []string{"WIP", "do not merge"}
+	if len(conditions.DoesNotHaveLabels) != len(expectedLabels) {
+		t.Errorf("Expected gh_pr_conditions.does_not_have_labels length=%d, but got %d", len(expectedLabels), len(conditions.DoesNotHaveLabels))
+	} else {
+		for i, label := range expectedLabels {
+			if conditions.DoesNotHaveLabels[i] != label {
+				t.Errorf("Expected gh_pr_conditions.does_not_have_labels[%d]=%q, but got %q", i, label, conditions.DoesNotHaveLabels[i])
+			}
+		}
+	}
+}

internal/gh/gh.go

@@ -121,6 +121,13 @@ func (g *Github) MatchesConditions(pr *github.PullRequest, prn cfg.PrNotificatio
 		prOlderThan := createdAt.Add(time.Duration(prn.Conditions.OlderThanSeconds) * time.Second)
 		isAfter := time.Now().After(prOlderThan)
 		if !isAfter {
+			// PR is not older than the specified time
+			return false
+		}
+	}
+	if len(prn.Conditions.DoesNotHaveLabels) > 0 {
+		if labelsMatched(pr.Labels, prn.Conditions.DoesNotHaveLabels) {
+			// If the PR has any of the labels in DoesNotHaveLabels, it doesn't match the conditions
 			return false
 		}
 	}

internal/gh/gh_test.go

@@ -19,13 +19,26 @@ func TestMatchesConditions_OlderThanSeconds(t *testing.T) {
 	// Condition: PR older than 1 hour
 	prn := cfg.PrNotification{
 		Conditions: cfg.PrConditions{
-			OlderThanSeconds: 3600, // 1 hour
+			OlderThanSeconds:  3600, // 1 hour
+			DoesNotHaveLabels: []string{"WIP"},
 		},
 	}
 
 	if !g.MatchesConditions(pr, prn) {
 		t.Error("Expected MatchesConditions to return true for PR older than OlderThanSeconds")
 	}
+
+	label := "WIP"
+	pr = &github.PullRequest{
+		CreatedAt: &github.Timestamp{Time: time.Now().Add(-2 * time.Hour)},
+		Labels: []*github.Label{
+			{Name: &label},
+		},
+	}
+
+	if g.MatchesConditions(pr, prn) {
+		t.Error("Expected MatchesConditions to return false for PR with labels in DoesNotHaveLabels")
+	}
 }
 
 func TestMatchesConditions_NotOlderThanSeconds(t *testing.T) {