provide additional information if reading a change file fails (#556)

miniscruff · Oct 21, 2023 · 18607a1 · 18607a1
1 parent d2a0b1f
commit 18607a1
Show file tree

Hide file tree

Showing 3 changed files with 15 additions and 10 deletions.
diff --git a/.changes/unreleased/Fixed-20231021-132322.yaml b/.changes/unreleased/Fixed-20231021-132322.yaml
@@ -0,0 +1,5 @@
+kind: Fixed
+body: Provide a better error message if a change file is invalid
+time: 2023-10-21T13:23:22.077665894-07:00
+custom:
+  Issue: "554"
diff --git a/.github/workflows/test.yml b/.github/workflows/test.yml
@@ -4,19 +4,19 @@ on:
   push:
     branches: [ main ]
     paths:
-    - 'cmd/'
-    - 'core/'
-    - 'shared/'
-    - 'then/'
+    - 'cmd/**'
+    - 'core/**'
+    - 'shared/**'
+    - 'then/**'
     - 'main.go'
     - 'go.sum'
   pull_request:
     branches: [ main ]
     paths:
-    - 'cmd/'
-    - 'core/'
-    - 'shared/'
-    - 'then/'
+    - 'cmd/**'
+    - 'core/**'
+    - 'shared/**'
+    - 'then/**'
     - 'main.go'
     - 'go.sum'
 

diff --git a/core/change.go b/core/change.go
@@ -420,12 +420,12 @@ func LoadChange(path string, rf shared.ReadFiler) (Change, error) {
 
 	bs, err := rf(path)
 	if err != nil {
-		return c, err
+		return c, fmt.Errorf("reading change file '%s': %w", path, err)
 	}
 
 	err = yaml.Unmarshal(bs, &c)
 	if err != nil {
-		return c, err
+		return c, fmt.Errorf("unmarshaling change file '%s': %w", path, err)
 	}
 
 	c.Filename = path