fix: make the messages more meaningful (#25)

* fix: make the messages more meaningful

* fix: update no files message

* fix: proper message when check has no files to check

* fix: update messages

* fix: drop redundant msg

src/cmds/js_cmds/apply.js

@@ -42,10 +42,15 @@ exports.handler = argv => {
  log.debug('rootDir?', root_dir)
  log.debug('codeFiles?', codeFiles)
 
- const prettyFiles = apply_fmt(jsFiles(codeFiles))
+ const js = jsFiles(codeFiles)
+ const prettyFiles = apply_fmt(js)
 
  if (prettyFiles.length === 0) {
- log.info('No files to style.')
+ if (js.length > 0) {
+ log.info(`${js.length} file(s) reformatted.`)
+ } else {
+ log.info('No files to format.')
+ }
  }
 
  configure(root_dir)

src/cmds/js_cmds/check.js

@@ -35,19 +35,25 @@ exports.handler = argv => {
  log.debug('rootDir?', root_dir)
  log.debug('codeFiles?', codeFiles)
 
- const prettyFiles = check_fmt(jsFiles(codeFiles))
+ const js = jsFiles(codeFiles)
+ const prettyFiles = check_fmt(js)
 
  const success = prettyFiles.length > 0
 
  if (success) {
  log.info(`Files to style:`)
  prettyFiles.forEach(f =>
- log.print(`\t${path.relative(process.cwd(), f)}`)
+ log.print(`${path.relative(process.cwd(), f)}`)
  )
- log.error('One or more files failed the style check')
+ log.print('')
+ log.error(`${prettyFiles.length} file(s) failed the style check`)
  process.exit(1)
  } else {
- log.info('No files to style.')
+ if (js.length > 0) {
+ log.info(`${js.length} file(s) passed the style check.`)
+ } else {
+ log.info('No files to check.')
+ }
  process.exit(0)
  }
 }

src/prettier.js

@@ -89,7 +89,6 @@ exports.apply_fmt = function apply_prettier(files) {
  ? log.debug('file written to disk')
  : log.debug('file write FAILED')
 
- log.info(`Reformatted: ${file}`)
  pretty_files.push(file)
  } catch (error) {
  log.error('Formatting failed.', file, error)