fix: include error message in write failure logs

AriPerkkio · Feb 7, 2021 · 84100b7 · 84100b7
1 parent bf8391b
commit 84100b7
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 5 deletions.
diff --git a/lib/index.ts b/lib/index.ts
@@ -112,7 +112,7 @@ async function scanRepo(repository: string) {
     try {
         writeResults(results, repository);
     } catch (e) {
-        logger.onWriteFailure(repository);
+        logger.onWriteFailure(repository, e);
     }
 
     logger.onLintEnd(repository, results.length);

diff --git a/lib/progress-logger/log-templates.ts b/lib/progress-logger/log-templates.ts
@@ -99,8 +99,8 @@ export const CLONE_FAILURE_TEMPLATE = (repository: string): string =>
 export const READ_FAILURE_TEMPLATE = (repository: string): string =>
     `[ERROR] ${repository} failed to read files`;
 
-export const WRITE_FAILURE_TEMPLATE = (repository: string): string =>
-    `[ERROR] ${repository} failed to write results`;
+export const WRITE_FAILURE_TEMPLATE = (repository: string, e: Error): string =>
+    `[ERROR] ${repository} failed to write results: ${e.message}`;
 
 export const OVERFLOWING_ROWS_TOP = (overflowingRowCount: number): string =>
     `[\u25B2 to see ${overflowingRowCount} lines above]`;

diff --git a/lib/progress-logger/progress-logger.ts b/lib/progress-logger/progress-logger.ts
@@ -375,9 +375,9 @@ class ProgressLogger {
     /**
      * Log error about result writing failure
      */
-    onWriteFailure(repository: string) {
+    onWriteFailure(repository: string, error: Error) {
         this.addNewMessage({
-            content: Templates.WRITE_FAILURE_TEMPLATE(repository),
+            content: Templates.WRITE_FAILURE_TEMPLATE(repository, error),
             color: 'red',
             level: 'error',
         });