Merge pull request #5 from gooddata/jimi/lint

fix(commit-lint): make commit-lint more robust
gooddata · Jun 17, 2024 · 63152f4 · 63152f4
2 parents e8737ac + 9bea2b8
commit 63152f4
Showing 1 changed file with 13 additions and 8 deletions.
diff --git a/.github/workflows/commit-lint.yaml b/.github/workflows/commit-lint.yaml
@@ -48,14 +48,19 @@ jobs:
                     const { type, subject } = parsedCommit;
                     if (type === 'chore' && subject?.startsWith('update')) return [true]; // skip renovate and extimage bumps
 
-                    const trailers = execSync('git interpret-trailers --parse', {
-                      input: parsedCommit.raw || '',
-                    }).toString();
-                    const matches = toLines(trailers).filter((ln) => ln.match(/^risk:\s*nonprod|low|high/i)).length
-                    return [
-                      matches === 1,
-                      `Should have exactly one risk label of value nonprod|low|high, but ${matches} found.`,
-                    ];
+                    try {
+                      const trailers = execSync('git interpret-trailers', ['--parse'], {
+                        input: parsedCommit.raw || '',
+                      }).toString(); 
+                      const matches = toLines(trailers)?.filter((ln) => ln.match(/^risk:\s*nonprod|low|high/i))?.length
+                      return [
+                        matches === 1,
+                        `Should have exactly one risk label of value nonprod|low|high, but ${matches} found.`,
+                      ];
+                    } catch (err) {
+                      console.error(err.toString());
+                      return [false, 'Error while trying to find risk label'];
+                    }
                   },
                 },
               },