Merge pull request #118066 from BenTheElder/yamlfmt-cleanup

yamlfmt cleanup

cmd/yamlfmt/OWNERS

@@ -1,4 +1,10 @@
 # See the OWNERS docs at https://go.k8s.io/owners
 
+reviewers:
+  - bentheelder
+  - dims
 approvers:
   - dims
+labels:
+  - sig/testing
+  - sig/contributor-experience

cmd/yamlfmt/yamlfmt.go

@@ -28,29 +28,26 @@ import (
 func main() {
 	indent := flag.Int("indent", 2, "default indent")
 	flag.Parse()
-
+	for _, path := range flag.Args() {
-	if flag.NArg() > 0 {
+		sourceYaml, err := os.ReadFile(path)
-		for _, path := range flag.Args() {
+		if err != nil {
-			sourceYaml, err := os.ReadFile(path)
+			fmt.Fprintf(os.Stderr, "%s: %v\n", path, err)
-			if err != nil {
+			continue
-				fmt.Fprintf(os.Stderr, "%s: %v\n", path, err)
+		}
-				continue
+		rootNode, err := fetchYaml(sourceYaml)
-			}
+		if err != nil {
-			rootNode, err := fetchYaml(sourceYaml)
+			fmt.Fprintf(os.Stderr, "%s: %v\n", path, err)
-			if err != nil {
+			continue
-				fmt.Fprintf(os.Stderr, "%s: %v\n", path, err)
+		}
-				continue
+		writer, err := os.OpenFile(path, os.O_WRONLY|os.O_CREATE|os.O_TRUNC, 0666)
-			}
+		if err != nil {
-			writer, err := os.OpenFile(path, os.O_WRONLY|os.O_CREATE|os.O_TRUNC, 0666)
+			fmt.Fprintf(os.Stderr, "%s: %v\n", path, err)
-			if err != nil {
+			continue
-				fmt.Fprintf(os.Stderr, "%s: %v\n", path, err)
+		}
-				continue
+		err = streamYaml(writer, indent, rootNode)
-			}
+		if err != nil {
-			err = streamYaml(writer, indent, rootNode)
+			fmt.Fprintf(os.Stderr, "%s: %v\n", path, err)
-			if err != nil {
+			continue
-				fmt.Fprintf(os.Stderr, "%s: %v\n", path, err)
-				continue
-			}
 		}
 	}
 }