@toptal/davinci-cli-shared 1.5.2-alpha-fx-2723-e2e-to-integration-migration-script.128 → 1.5.2-alpha-fx-2815-trailing-comma--eslint.127

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@toptal/davinci-cli-shared",
-  "version": "1.5.2-alpha-fx-2723-e2e-to-integration-migration-script.128+0c76c7cb",
+  "version": "1.5.2-alpha-fx-2815-trailing-comma--eslint.127+ab9fe716",
   "description": "Shared CLI code and CLI engine for davinci",
   "publishConfig": {
     "access": "public"
@@ -20,5 +20,5 @@
     "js-yaml": "^4.1.0",
     "vorpal": "^1.12.0"
   },
-  "gitHead": "0c76c7cbaa05e3fdfe6a6baea58740b84123b2b9"
+  "gitHead": "ab9fe71658d458c63ac71de4b1ce828507b8487e"
 }

package/src/commands.loader.js

@@ -12,7 +12,9 @@ const commandsLoader = (commandCreators, vorpalInstance) => {
 
     const commandInstance = vorpalInstance
       .command(command)
-      .parse((command, args) => command.replace(args, toVorpalArgs(args)))
+      .parse((commandText, args) =>
+        commandText.replace(args, toVorpalArgs(args))
+      )
 
     options.forEach(option => commandInstance.option(option.name, option.label))
 

package/src/index.js

@@ -3,7 +3,9 @@ const vorpal = require('vorpal')
 const print = require('../src/utils/print')
 const { run, runSync } = require('../src/utils/run')
 const prompt = require('../src/utils/prompt')
-const { convertToCLIParameters } = require('../src/utils/convert-to-cli-parameters')
+const {
+  convertToCLIParameters
+} = require('../src/utils/convert-to-cli-parameters')
 const davinciProjectConfig = require('./utils/davinci-project-config')
 const files = require('./utils/files')
 
@@ -17,11 +19,12 @@ const execCommand = (...args) => {
   return vorpalInstance.exec(...args)
 }
 
-const help = help => {
-  return vorpalInstance.help(help)
+const help = helpText => {
+  return vorpalInstance.help(helpText)
 }
 
 const catchErrorCommand = callback => {
+  /* eslint-disable promise/catch-or-return, promise/valid-params */
   vorpalInstance
     .catch('[commands...]', 'Catches incorrect commands')
     .action((args, cb) => {

package/src/utils/convert-to-cli-parameters.js

@@ -1,11 +1,13 @@
 const convertToCLIParameters = options => {
   const result = []
+
   Object.keys(options).forEach(key => {
-      result.push(`--${key}`)
+    result.push(`--${key}`)
+
+    const hasTrueValue = options[key] === true
+    const optionValue = hasTrueValue ? '' : String(options[key])
 
-      const hasTrueValue = options[key] === true
-      const optionValue = hasTrueValue ? '' : String(options[key])
-      result.push(optionValue)
+    result.push(optionValue)
   })
 
   return result

package/src/utils/davinci-project-config.js

@@ -1,10 +1,12 @@
 const fs = require('fs')
 const yaml = require('js-yaml')
+
 const files = require('./files')
 
 const path = files.getProjectRootFilePath('./davinci.yaml')
 const data = fs.existsSync(path) ? yaml.load(fs.readFileSync(path, 'utf8')) : {}
-const resolve = (sectionName, key) => Boolean(data && resolveKey(data[sectionName], key))
+const resolve = (sectionName, key) =>
+  Boolean(data && resolveKey(data[sectionName], key))
 const resolveKey = (section, key) => section && section[key]
 
 const davinciProjectConfig = {

package/src/utils/files.js

@@ -1,23 +1,21 @@
 const path = require('path')
 
 const getPackageFileContent = (davinciPackageName, filename) =>
-  require(
-    getPackageFilePath(davinciPackageName, filename)
-  )
+  require(getPackageFilePath(davinciPackageName, filename))
 
 const getPackageFilePath = (davinciPackageName, filename) => {
   const packageIndexPath = require.resolve(davinciPackageName)
   const davinciQARoot = path.join(packageIndexPath, '../..')
+
   return path.join(davinciQARoot, filename)
 }
 
 const getProjectRootFileContent = filename =>
-  require(
-    getProjectRootFilePath(filename)
-  )
+  require(getProjectRootFilePath(filename))
 
 const getProjectRootFilePath = filename => {
   const currentRunningDir = process.cwd()
+
   return path.join(currentRunningDir, filename)
 }