fix: wrap errors returned by JSON unmarshal

ThatsMrTalbot · Adam Talbot · commit 4c6913f457f3 · 2025-03-14T10:37:11.000-07:00
Currently `%v` is used inside an `Errorf` call, this means the original errors is lost and cannot be unwrapped to. 

By changing it to `%w` the error is preserved and can be retrieved with `errors.As`.
diff --git a/yaml.go b/yaml.go
@@ -92,7 +92,7 @@ func jsonUnmarshal(reader io.Reader, obj interface{}, opts ...JSONOpt) error {
 		d = opt(d)
 	}
 	if err := d.Decode(&obj); err != nil {
-		return fmt.Errorf("while decoding JSON: %v", err)
+		return fmt.Errorf("while decoding JSON: %w", err)
 	}
 	return nil
 }

Original file line number	Diff line number	Diff line change
`@@ -92,7 +92,7 @@ func jsonUnmarshal(reader io.Reader, obj interface{}, opts ...JSONOpt) error {`
`92`	`92`	`d = opt(d)`
`93`	`93`	`}`
`94`	`94`	`if err := d.Decode(&obj); err != nil {`
`95`		`- return fmt.Errorf("while decoding JSON: %v", err)`
	`95`	`+ return fmt.Errorf("while decoding JSON: %w", err)`
`96`	`96`	`}`
`97`	`97`	`return nil`
`98`	`98`	`}`