Add ci.minSeverity

cmd/pint/bench_test.go

@@ -102,6 +102,6 @@ rule {
 
 	b.ResetTimer()
 	for n := 0; n < b.N; n++ {
-		_, _ = checkRules(ctx, 10, false, gen, cfg, entries)
+		_, _ = checkRules(ctx, "", 10, false, gen, cfg, entries)
 	}
 }

cmd/pint/ci.go

@@ -108,7 +108,7 @@ func actionCI(c *cli.Context) error {
 
 	slog.Debug("Generated all Prometheus servers", slog.Int("count", gen.Count()))
 
-	summary, err := checkRules(ctx, meta.workers, meta.isOffline, gen, meta.cfg, entries)
+	summary, err := checkRules(ctx, "ci", meta.workers, meta.isOffline, gen, meta.cfg, entries)
 	if err != nil {
 		return err
 	}

cmd/pint/lint.go

@@ -82,7 +82,7 @@ func actionLint(c *cli.Context) error {
 		return err
 	}
 
-	summary, err := checkRules(ctx, meta.workers, meta.isOffline, gen, meta.cfg, entries)
+	summary, err := checkRules(ctx, "lint", meta.workers, meta.isOffline, gen, meta.cfg, entries)
 	if err != nil {
 		return err
 	}

cmd/pint/scan.go

@@ -28,7 +28,7 @@ const (
 This usually means that it's missing some required fields.`
 )
 
-func checkRules(ctx context.Context, workers int, isOffline bool, gen *config.PrometheusGenerator, cfg config.Config, entries []discovery.Entry) (summary reporter.Summary, err error) {
+func checkRules(ctx context.Context, cmd string, workers int, isOffline bool, gen *config.PrometheusGenerator, cfg config.Config, entries []discovery.Entry) (summary reporter.Summary, err error) {
 	if isOffline {
 		slog.Info("Offline mode, skipping Prometheus discovery")
 	} else {
@@ -127,9 +127,26 @@ func checkRules(ctx context.Context, workers int, isOffline bool, gen *config.Pr
 		defer close(jobs)
 	}()
 
+	shouldReport := func(sev checks.Severity) bool {
+		return true
+	}
+
+	if cmd == "ci" {
+		minSev, err := checks.ParseSeverity(cfg.CI.MinSeverity)
+		if err != nil {
+			minSev = checks.Warning
+		}
+		shouldReport = func(sev checks.Severity) bool {
+			return sev >= minSev
+		}
+	}
+
 	for result := range results {
-		summary.Report(result)
+		if shouldReport(result.Problem.Severity) {
+			summary.Report(result)
+		}
 	}
+
 	summary.SortReports()
 	summary.Duration = time.Since(start)
 	summary.TotalEntries = len(entries)

cmd/pint/watch.go

@@ -317,7 +317,7 @@ func (c *problemCollector) scan(ctx context.Context, workers int, isOffline bool
 		return err
 	}
 
-	s, err := checkRules(ctx, workers, isOffline, gen, c.cfg, entries)
+	s, err := checkRules(ctx, "watch", workers, isOffline, gen, c.cfg, entries)
 	if err != nil {
 		return err
 	}

internal/config/ci.go

@@ -2,16 +2,23 @@ package config
 
 import (
 	"errors"
+	"fmt"
+
+	"github.com/cloudflare/pint/internal/checks"
 )
 
 type CI struct {
-	BaseBranch string `hcl:"baseBranch,optional" json:"baseBranch,omitempty"`
-	MaxCommits int    `hcl:"maxCommits,optional" json:"maxCommits,omitempty"`
+	BaseBranch  string `hcl:"baseBranch,optional" json:"baseBranch,omitempty"`
+	MaxCommits  int    `hcl:"maxCommits,optional" json:"maxCommits,omitempty"`
+	MinSeverity string `hcl:"minSeverity,optional" json:"minSeverity,omitempty"`
 }
 
 func (ci CI) validate() error {
 	if ci.MaxCommits <= 0 {
 		return errors.New("maxCommits cannot be <= 0")
 	}
+	if _, err := checks.ParseSeverity(ci.MinSeverity); err != nil {
+		return fmt.Errorf("invalid minSeverity: %w", err)
+	}
 	return nil
 }

internal/config/config.go

@@ -222,8 +222,9 @@ func getContext() *hcl.EvalContext {
 func Load(path string, failOnMissing bool) (cfg Config, fromFile bool, err error) {
 	cfg = Config{
 		CI: &CI{
-			MaxCommits: 20,
-			BaseBranch: "master",
+			MaxCommits:  20,
+			BaseBranch:  "master",
+			MinSeverity: "info",
 		},
 		Parser: &Parser{},
 		Checks: &Checks{