fix(cli): do not exit after 1st formatted file

terrastruct · Aug 2, 2023 · a4caac4 · a4caac4
1 parent 339c0b3
commit a4caac4
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 1 deletion.
diff --git a/ci/release/changelogs/next.md b/ci/release/changelogs/next.md
@@ -3,3 +3,5 @@
 #### Improvements 🧹
 
 #### Bugfixes ⛑️
+
+- Fixes `d2 fmt` to format all files passed as arguments rather than first non-formatted only [#1523](https://github.com/terrastruct/d2/issues/1523)
diff --git a/d2cli/fmt.go b/d2cli/fmt.go
@@ -33,7 +33,9 @@ func fmtCmd(ctx context.Context, ms *xmain.State) (err error) {
 
 		output := []byte(d2format.Format(m))
 		if !bytes.Equal(output, input) {
-			return ms.WritePath(inputPath, output)
+			if err := ms.WritePath(inputPath, output); err != nil {
+				return err
+			}
 		}
 	}
 	return nil