Merge pull request #1505 from alixander/ignore-debug

ignore invalid debug flag
terrastruct · Jul 29, 2023 · 9ac7f3d · 9ac7f3d
2 parents 0f7dc63 + 477014d
commit 9ac7f3d
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 1 deletion.
diff --git a/ci/release/changelogs/next.md b/ci/release/changelogs/next.md
@@ -16,6 +16,7 @@
 - Tooltip and link icons are now positioned on shape border [#1466](https://github.com/terrastruct/d2/pull/1466)
 - Tooltip and link icons are always rendered over shapes [#1467](https://github.com/terrastruct/d2/pull/1467)
 - Boards with no objects are considered folders [#1504](https://github.com/terrastruct/d2/pull/1504)
+- `DEBUG` environment variable ignored if set incorrectly [#1505](https://github.com/terrastruct/d2/pull/1505)
 
 #### Bugfixes ⛑️
 

diff --git a/d2cli/main.go b/d2cli/main.go
@@ -67,7 +67,8 @@ func Run(ctx context.Context, ms *xmain.State) (err error) {
 	}
 	debugFlag, err := ms.Opts.Bool("DEBUG", "debug", "d", false, "print debug logs.")
 	if err != nil {
-		return err
+		ms.Log.Warn.Printf("Invalid DEBUG flag value ignored")
+		debugFlag = go2.Pointer(false)
 	}
 	imgCacheFlag, err := ms.Opts.Bool("IMG_CACHE", "img-cache", "", true, "in watch mode, images used in icons are cached for subsequent compilations. This should be disabled if images might change.")
 	if err != nil {