fixing issues with json reporting and grouping

Boeing · Dec 18, 2024 · 4bf5b9e · 4bf5b9e
1 parent 7308484
commit 4bf5b9e
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 5 deletions.
diff --git a/cmd/validator/validator.go b/cmd/validator/validator.go
@@ -238,8 +238,6 @@ func validateGroupByConf(groupBy *string) error {
 
 func validateGlobbing(globbingPrt *bool) error {
 	if *globbingPrt && (isFlagSet("exclude-dirs") || isFlagSet("exclude-file-types")) {
-		fmt.Println("The -globbing flag cannot be used with --exclude-dirs or --exclude-file-types")
-		flag.Usage()
 		return errors.New("the -globbing flag cannot be used with --exclude-dirs or --exclude-file-types")
 	}
 	return nil

diff --git a/pkg/cli/cli.go b/pkg/cli/cli.go
@@ -151,7 +151,7 @@ func (c CLI) printGroupSingle(reports []reporter.Report) error {
 
 	// Check reporter type to determine how to print
 	for _, reporterObj := range c.Reporters {
-		if _, ok := reporterObj.(reporter.JSONReporter); ok {
+		if _, ok := reporterObj.(*reporter.JSONReporter); ok {
 			return reporter.PrintSingleGroupJSON(reportGroup)
 		}
 	}
@@ -167,7 +167,7 @@ func (c CLI) printGroupDouble(reports []reporter.Report) error {
 
 	// Check reporter type to determine how to print
 	for _, reporterObj := range c.Reporters {
-		if _, ok := reporterObj.(reporter.JSONReporter); ok {
+		if _, ok := reporterObj.(*reporter.JSONReporter); ok {
 			return reporter.PrintDoubleGroupJSON(reportGroup)
 		}
 	}
@@ -182,7 +182,7 @@ func (c CLI) printGroupTriple(reports []reporter.Report) error {
 	}
 
 	for _, reporterObj := range c.Reporters {
-		if _, ok := reporterObj.(reporter.JSONReporter); ok {
+		if _, ok := reporterObj.(*reporter.JSONReporter); ok {
 			return reporter.PrintTripleGroupJSON(reportGroup)
 		}
 	}