Updates (particularly for spdx-correct and spdx-satisfies)

.eslintignore

@@ -0,0 +1 @@
+node_modules

.travis.yml

@@ -1,4 +1,16 @@
 language: node_js
 node_js:
  - "node"
- - "8"
+ - "8.3"
+ - "10"
+ - "12"
+before_script: >
+ node_version=$(node -v);
+ if [ ${node_version:3:1} = "." ]; then
+ echo "Node 10+"
+ else
+ echo "Node 8"
+ npm install --no-save "eslint@5"
+ fi
+script:
+ - npm run test

index.js

@@ -3,15 +3,15 @@
 */
 
 const program = require('commander');
-const packageJson = require('./package.json');
 const moment = require('moment');
 const inquirer = require('inquirer');
+const colors = require('colors/safe');
+const packageJson = require('./package.json');
 const getPackageDetails = require('./lib/getPackageDetails');
 const walkDependencies = require('./lib/walkDependencies');
 const showImpact = require('./lib/showImpact');
 const showDetails = require('./lib/showDetails');
 const showQuickStats = require('./lib/showQuickStats');
-const colors = require('colors/safe');
 const install = require('./lib/install');
 const exec = require('./lib/exec');
 const getInstallCommand = require(

lib/calculateImpactPackages.js

@@ -14,7 +14,7 @@ function flatingPackages(packages) {
  let duplicated = false;
  for (let i = 0; i < flatPackages.length; i += 1) {
  const key = flatPackages[i];
- const name = packages[key].name;
+ const { name } = packages[key];
  if (name === newName) {
  duplicated = true;
  break;

lib/exec.js

@@ -2,7 +2,7 @@
  * @file exec command with args
  */
 
-const spawn = require('child_process').spawn;
+const { spawn } = require('child_process');
 
 /**
  * @param {string} command

lib/getInstallCommand.js

@@ -2,8 +2,8 @@
  * @file get install command considering package manager and arguments
  */
 
-const testYarn = require('./testYarn');
 const program = require('commander');
+const testYarn = require('./testYarn');
 const isProduction = require('./isProduction');
 
 /**

lib/getLicenseStr.js

@@ -5,9 +5,11 @@
 module.exports = function getLicenseStr(licenseObj) {
  if (typeof licenseObj === 'string') {
  return licenseObj;
- } else if (Array.isArray(licenseObj)) {
+ }
+ if (Array.isArray(licenseObj)) {
  return `(${licenseObj.map(getLicenseStr).join(` OR `)})`;
- } else if (typeof licenseObj === 'object') {
+ }
+ if (typeof licenseObj === 'object') {
  return licenseObj.type || `Unknown`;
  }
  return `Unknown`;

lib/getLocalPackage.js

@@ -2,9 +2,9 @@
  * @file get local package.json
  */
 
-const findPrefixPromise = require('./findPrefixPromise');
 const fs = require('fs');
 const path = require('path');
+const findPrefixPromise = require('./findPrefixPromise');
 
 /**
  * @return {Promise.<Object>} package.json

lib/getPackageDetails.js

@@ -4,14 +4,15 @@
 
 const fetch = require('node-fetch');
 const semver = require('semver');
+const url = require('url');
+const readline = require('readline');
+const rc = require('rc');
 const getLicenseStr = require('./getLicenseStr');
 const getLicenseType = require('./getLicenseType');
-const url = require('url');
 const printError = require('./printError');
-const readline = require('readline');
 
 const packageDetailsCache = {};
-const npmConfig = require('rc')('npm', {
+const npmConfig = rc('npm', {
  registry: `https://registry.npmjs.org/`
 });
 
@@ -108,7 +109,9 @@ function getPackageJsonFromGitHub(owner, repo, ref) {
  * @param {string} versionLoose
  * @returns {object} details like dependencies, version, size, license, modified
  */
-function getPackageDetailsFromGitHub({ host, path, hash, protocol }, versionLoose) {
+function getPackageDetailsFromGitHub({
+ host, path, hash, protocol
+}, versionLoose) {
  let owner;
  let repo;
  if (protocol === 'github:') {

lib/getPackagesStats.js

@@ -26,5 +26,7 @@ module.exports = function getPackagesStats(packages) {
  }
  licenseTypes[type] += licenses[license];
  });
- return { count, size, licenses, licenseTypes };
+ return {
+ count, size, licenses, licenseTypes
+ };
 };

lib/install.js

@@ -1,20 +1,20 @@
 /**
  * @file install action
  */
-const getLocalPackage = require('./getLocalPackage');
 const colors = require('colors/safe');
+const inquirer = require('inquirer');
+const filesize = require('filesize');
+const program = require('commander');
+const readline = require('readline');
+const getLocalPackage = require('./getLocalPackage');
 const walkDependencies = require('./walkDependencies');
 const getInstallCommand = require('./getInstallCommand');
-const inquirer = require('inquirer');
 const exec = require('./exec');
 const showDetails = require('./showDetails');
 const isProduction = require('./isProduction');
 const getPackagesStats = require('./getPackagesStats');
 const formatLicenseType = require('./formatLicenseType');
-const filesize = require('filesize');
 const getSimpleTable = require('./getSimpleTable');
-const program = require('commander');
-const readline = require('readline');
 const printError = require('./printError');
 
 /**

lib/showImpact.js

@@ -3,14 +3,14 @@
 */
 
 const program = require('commander');
+const filesize = require('filesize');
+const readline = require('readline');
 const getLocalPackage = require('./getLocalPackage');
 const calculateImpactPackages = require('./calculateImpactPackages');
 const satisfies = require('./satisfies');
 const walkDependencies = require('./walkDependencies');
 const getPackagesStats = require('./getPackagesStats');
-const filesize = require('filesize');
 const getSimpleTable = require('./getSimpleTable');
-const readline = require('readline');
 
 /**
  * @param {Object} impactLicenses license -> count
@@ -67,9 +67,9 @@ module.exports = function showImpact(
  const devDependencies = localPackage.devDependencies || {};
  let allDependencies;
  if (program.args[1].saveDev) {
- allDependencies = Object.assign({}, dependencies, devDependencies);
+ allDependencies = { ...dependencies, ...devDependencies };
  } else {
- allDependencies = Object.assign({}, dependencies);
+ allDependencies = { ...dependencies };
  }
  if (Object.keys(allDependencies).length === 0) {
  console.log('Local package has no dependencies');

lib/showQuickStats.js

@@ -2,11 +2,11 @@
  * @file print quick stats
  */
 
+const filesize = require('filesize');
+const readline = require('readline');
 const getPackagesStats = require('./getPackagesStats');
 const formatLicenseType = require('./formatLicenseType');
-const filesize = require('filesize');
 const getSimpleTable = require('./getSimpleTable');
-const readline = require('readline');
 
 /**
  * @param {Object} packages

lib/testYarn.js

@@ -2,9 +2,9 @@
  * @file detects if package initialised with yarn
  */
 
-const findPrefixPromise = require('./findPrefixPromise');
 const fs = require('fs');
 const path = require('path');
+const findPrefixPromise = require('./findPrefixPromise');
 
 module.exports = function testYarn() {
  return findPrefixPromise().then((packagePath) => {