fix: improve logging for common situations

dhis2 · Nov 22, 2019 · 0b435db · 0b435db
1 parent ab63b9b
commit 0b435db
Show file tree

Hide file tree

Showing 2 changed files with 14 additions and 2 deletions.
diff --git a/src/commands/javascript.js b/src/commands/javascript.js
@@ -39,7 +39,11 @@ const handler = (argv, apply) => {
     opts.files = selectFiles(files, pattern, staged)
 
     if (opts.files.length === 0) {
-        log.debug('No files to work on...')
+        log.warn(
+            `No matching ${
+                staged ? 'staged ' : ''
+            }files for pattern "${pattern}"`
+        )
         return
     }
 
@@ -54,6 +58,8 @@ const handler = (argv, apply) => {
         config: prettierConfig,
         ...opts,
     })
+
+    log.info(`${opts.files.length} file(s) ${apply ? 'fixed' : 'checked'}`)
 }
 
 const javascriptCmds = yargs => {

diff --git a/src/commands/structured-text.js b/src/commands/structured-text.js
@@ -34,7 +34,11 @@ const handler = (argv, apply) => {
     opts.files = selectFiles(files, pattern, staged)
 
     if (opts.files.length === 0) {
-        log.debug('No files to work on...')
+        log.warn(
+            `No matching ${
+                staged ? 'staged ' : ''
+            }files for pattern "${pattern}"`
+        )
         return
     }
 
@@ -44,6 +48,8 @@ const handler = (argv, apply) => {
         config: prettierConfig,
         ...opts,
     })
+
+    log.info(`${opts.files.length} file(s) ${apply ? 'fixed' : 'checked'}`)
 }
 
 const textCmds = yargs => {