This fixes set-exit-status when using ./...

#93
incu6us · May 27, 2024 · 9db37ec · 9db37ec
1 parent 5f29621
commit 9db37ec
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 0 deletions.
diff --git a/main.go b/main.go
@@ -275,6 +275,9 @@ func main() {
 				log.Fatalf("Failed to find unformatted files %s: %+v\n", originPath, err)
 			}
 			fmt.Printf("%s\n", unformattedFiles.String())
+			if *setExitStatus && unformattedFiles != nil {
+				os.Exit(1)
+			}
 			return
 		}
 		err := reviser.NewSourceDir(originProjectName, originPath, *isRecursive, excludes).Fix(options...)

diff --git a/reviser/dir.go b/reviser/dir.go
@@ -121,6 +121,10 @@ func (d *SourceDir) Find(options ...SourceFileOption) (*UnformattedCollection, e
 		return nil, fmt.Errorf("failed to walk dif: %w", err)
 	}
 
+	if len(badFormattedCollection) == 0 {
+		return nil, nil
+	}
+
 	return newUnformattedCollection(badFormattedCollection), nil
 }
 
@@ -176,6 +180,10 @@ func (c *UnformattedCollection) List() []string {
 }
 
 func (c *UnformattedCollection) String() string {
+	if c == nil {
+		return ""
+	}
+
 	var builder strings.Builder
 	for i, file := range c.list {
 		builder.WriteString(file)