chore: Removed unneeded packages from modules and package file

Author: ajfisher

bin/cli.js

@@ -5,6 +5,7 @@ const program = require('commander');
 const version = require('../package.json').version;
 const Interchange = require('../lib/interchange');
 const Inquire = require('../lib/inquire');
+
 const interchange = new Interchange();
 
 program

lib/interchange.js

@@ -1,10 +1,8 @@
 const _ = require('lodash');
-const program = require('commander');
 const Avrgirl = require('avrgirl-arduino');
 const Serialport = require('serialport');
 const child_process = require('child_process');
 const colors = require('colors');
-const fs      = require('fs');
 const fsextra = require('fs-extra');
 
 const interchange_client = require('./interchange_client');
@@ -16,7 +14,6 @@ const version = require('../package.json').version;
 
 // I think this can be refactored to be purely in the function where it is needed
 let ic_client;//  = new interchange_client.Client();
-// let fw = null; // used to hold details of the firmware we want in it.
 
 const Interchange = function() {
   this.firmwares = firmwares;
@@ -151,7 +148,6 @@ Interchange.prototype.flash_firmware = function(firmware, opts) {
 
     avrgirl.flash(firmware, (err) => {
       if (err) {
-        // console.error(err);
         reject(err);
       }
 
@@ -181,25 +177,22 @@ Interchange.prototype.install_firmware = async function(firmware, options = {})
   // check firmware
   // download firmware
   // flash firmware to device
-  // optionally do interchange client things if needed
+  // optionally do interchange client configuration
 
   try {
-    /* istanbul ignore next */
-    if (!process.env.TEST) {
-      console.log('about to check firmware');
-    }
+    // check the firmware
     const {fw, opts} = await this.check_firmware(firmware, settings);
     const usingFirmata = opts.useFirmata || false; // Assumes not unless explicit
 
     /* istanbul ignore next */
     if (!process.env.TEST) {
-      console.log('now download the firmware');
+      console.log('Downloading firmware');
     }
     const {hexpath, tmpdir} = await this.download_firmware(fw, opts);
 
     /* istanbul ignore next */
     if (!process.env.TEST) {
-      console.log('now flash the firmware');
+      console.log('Flashing firmware to board');
     }
     const port = await this.flash_firmware(hexpath, options)
       .then((serport) => {
@@ -216,7 +209,7 @@ Interchange.prototype.install_firmware = async function(firmware, options = {})
     if (!usingFirmata) {
       /* istanbul ignore next */
       if (!process.env.TEST) {
-        console.log('About to configure the firmware'.magenta);
+        console.log('Configuring the firmware'.magenta);
       }
       // combine options and firmware details together and pass across
       return this.set_firmware_details(port, {...opts, ...fw})
@@ -240,7 +233,6 @@ Interchange.prototype.get_firmware_info = function(port) {
     ic_client.port = port;
 
     ic_client.on('error', (err) => {
-      // console.error('ic_client error', err);
       reject(err);
     });
 
@@ -313,7 +305,6 @@ Interchange.prototype.set_firmware_details = (port, opts) => {
       ic_client.port = port;
 
       ic_client.on('error', (err) => {
-        // console.error("Can't config device. Did you remember to set your backpack into config mode?".red);
         ic_client.close();
         reject(err);
       });

package-lock.json

@@ -28,8 +28,8 @@
   },
   "husky": {
     "hooks": {
-        "pre-commit": "npm test",
-        "pre-push": "npm test"
+      "pre-commit": "npm test",
+      "pre-push": "npm test"
     }
   },
   "repository": {
@@ -65,15 +65,13 @@
     "fs-extra": "^8.1.0",
     "inquirer": "^7.0.0",
     "lodash": "^4.17.15",
-    "minimist": "^1.2.0",
     "serialport": "^8.0.5",
     "tmp": "^0.1.0"
   },
   "devDependencies": {
     "eslint": "^6.7.2",
     "husky": "^3.1.0",
     "jest": "^24.9.0",
-    "mock-stdin": "^1.0.0",
     "standard-version": "^7.0.1"
   }
 }