Removed the summary of violations at the end since we are printing th…

…em as they occur

pkg/safeguards/safeguards.go

@@ -63,24 +63,24 @@ func ValidateManifests(ctx context.Context, manifestFiles []string) error {
 
 	var violations []string
 	for _, m := range manifestFiles {
+		var tempViolations []string
 		manifests, err := fc.ReadManifests(m)
 		if err != nil {
 			log.Errorf("reading manifests %s", err.Error())
 			return err
 		}
 
 		// validation of deployment manifest with constraints, templates loaded
-		violations, err = validateManifests(ctx, c, manifests)
+		tempViolations, err = validateManifests(ctx, c, manifests)
 		if err != nil {
 			log.Errorf("validating manifests: %s", err.Error())
 			return err
 		}
+		violations = append(violations, tempViolations...)
 	}
 
 	// returning the full list of violations after each manifest is checked
-	if len(violations) > 0 {
-		log.Printf("violations have occurred: %s", violations)
-	} else {
+	if len(violations) == 0 {
 		log.Printf("No violations found.")
 	}
 

pkg/safeguards/safeguards_helpers.go

@@ -180,7 +180,7 @@ func validateManifests(ctx context.Context, c *constraintclient.Client, manifest
 				}
 			}
 		}
-		log.Printf("\n")
+		fmt.Println("\n")
 	}
 
 	return violations, nil