feat: make rule builder return all violations instead of just the one…

… that matches the first should
omissis · Aug 8, 2022 · 026e4f6 · 026e4f6
1 parent 97ed085
commit 026e4f6
Showing 1 changed file with 4 additions and 4 deletions.
diff --git a/internal/arch/file/rule.go b/internal/arch/file/rule.go
@@ -68,13 +68,13 @@ func (rb *RuleBuilder) Because(b rule.Because) []rule.Violation {
 	}
 
 	for _, should := range rb.shoulds {
-		rb.Violations = should.Evaluate(rb)
-		if len(rb.Violations) > 0 {
-			return rb.Violations
+		vs := should.Evaluate(rb)
+		if len(vs) > 0 {
+			rb.Violations = append(rb.Violations, vs...)
 		}
 	}
 
-	return nil
+	return rb.Violations
 }
 
 func (rb *RuleBuilder) GetFiles() []string {