feat: drop support for Node <8.3, drop standard, upgrade deps

Author: niftylettuce

.npmignore

@@ -1,3 +1 @@
 media
-.nyc_output
-coverage

README.md

@@ -50,7 +50,7 @@
 
 * Unit testing with [ava][]
 * Commit linting with [commitlint][]
-* Linting with [eslint][] (with an option to choose between [prettier][] and [xo][] or [standard][])
+* Linting with [xo][] (uses [eslint][] and [prettier][])
 * Markdown Linting with [remark][]
 * Automatic code formatting with [prettier][]
 * Automatic `git init`
@@ -223,8 +223,6 @@ If you are seeking permission to use these trademarks, then please [contact us](
 
 [express]: https://expressjs.com
 
-[standard]: https://standardjs.com/
-
 [commitlint]: https://github.com/marionebl/commitlint
 
 [np]: https://github.com/sindresorhus/np

package.json

@@ -17,6 +17,9 @@
       "template/test/**/*.js",
       "template/**/__tests__/**/*.js",
       "template/**/*.test.js"
+    ],
+    "helpers": [
+      "template/test/helpers/**/*"
     ]
   },
   "bin": "cli.js",
@@ -35,43 +38,43 @@
     "Shaun Warman <[email protected]> (http://shaunwarman.com)"
   ],
   "dependencies": {
-    "cac": "^6.3.12",
-    "camelcase": "^5.0.0",
-    "execa": "^1.0.0",
+    "cac": "^6.5.2",
+    "camelcase": "^5.3.1",
+    "execa": "^2.0.4",
     "fixpack": "^2.3.1",
-    "github-username": "^4.1.0",
+    "github-username": "^5.0.1",
     "github-username-regex": "^1.0.0",
-    "global": "^4.3.2",
+    "global": "^4.4.0",
     "is-email": "^1.0.0",
     "is-url": "^1.2.4",
     "is-valid-npm-name": "^0.0.4",
     "npm-conf": "^1.1.3",
     "sao": "0.22.17",
-    "semver": "^5.6.0",
+    "semver": "^6.3.0",
     "shelljs": "^0.8.3",
-    "spdx-license-list": "^5.0.0",
+    "spdx-license-list": "^6.1.0",
     "speakingurl": "^14.0.1",
-    "superb": "^3.0.0",
-    "update-notifier": "^2.5.0",
+    "superb": "^4.0.0",
+    "update-notifier": "^3.0.1",
     "uppercamelcase": "^3.0.0"
   },
   "devDependencies": {
-    "@commitlint/cli": "^7.2.1",
-    "@commitlint/config-conventional": "^7.1.2",
-    "ava": "^1.0.1",
-    "codecov": "^3.1.0",
-    "cross-env": "^5.2.0",
-    "eslint": "^5.12.0",
+    "@commitlint/cli": "^8.1.0",
+    "@commitlint/config-conventional": "^8.1.0",
+    "ava": "^2.3.0",
+    "codecov": "^3.5.0",
+    "cross-env": "^5.2.1",
+    "eslint": "^6.3.0",
     "eslint-config-xo-lass": "^1.0.3",
-    "husky": "^1.3.1",
-    "lint-staged": "^8.1.0",
-    "nyc": "^13.1.0",
-    "remark-cli": "^6.0.1",
-    "remark-preset-github": "^0.0.13",
-    "xo": "^0.23.0"
+    "husky": "^3.0.5",
+    "lint-staged": "^9.2.5",
+    "nyc": "^14.1.1",
+    "remark-cli": "^7.0.0",
+    "remark-preset-github": "^0.0.16",
+    "xo": "^0.24.0"
   },
   "engines": {
-    "node": ">=7.6.0"
+    "node": ">=8.3.0"
   },
   "homepage": "https://lass.js.org",
   "husky": {

sao.js

@@ -67,13 +67,6 @@ module.exports = {
       default: conf.get('init-version') || '0.0.0',
       validate: val => (semver.valid(val) ? true : 'Invalid semver version')
     },
-    eslint: {
-      message: 'Choose an eslint configuration',
-      choices: ['prettier', 'standard'],
-      type: 'list',
-      default: 'prettier',
-      store: true
-    },
     author: {
       message: "What is your name (the author's)",
       default: conf.get('init-author-name') || ':gitUser:',
@@ -201,9 +194,8 @@ module.exports = {
       ctx.npmInstall();
     }
 
-    // Format code according to eslint configuration
-    const linter = ctx.answers.eslint === 'standard' ? 'standard' : 'xo';
-    await execa(`./node_modules/.bin/${linter}`, ['--fix'], {
+    // Format code with xo
+    await execa(`./node_modules/.bin/xo`, ['--fix'], {
       cwd: ctx.folderPath,
       stdio: 'inherit'
     });

template/.npmignore

@@ -1,2 +1 @@
-.nyc_output
-coverage
+media

template/index.js

@@ -1,6 +1,6 @@
 class Script {
   constructor(config) {
-    config = Object.assign({}, config);
+    config = { ...config };
     this._name = config.name || 'script';
 
     this.renderName = this.renderName.bind(this);

template/package

@@ -24,19 +24,14 @@
     "codecov": "latest",
     "cross-env": "latest",
     "eslint": "latest",
-    "eslint-config-<%- eslint %>": "latest",
-    "eslint-plugin-<%- eslint %>": "latest",
     "fixpack": "latest",
     "husky": "latest",
     "lint-staged": "latest",
     "nyc": "latest",
-    "prettier": "latest",
     "remark-cli": "latest",
     "remark-preset-github": "latest",
-    <% if (eslint === 'prettier') { %>
     "eslint-config-xo-lass": "latest",
-    <% } %>
-    <%- eslint === 'standard' ? `"standard"`: `"xo"` %>: "latest"
+    "xo": "latest"
   },
   "engines": {
     "node": ">=8.3"
@@ -51,18 +46,12 @@
     }
   },
   "lint-staged": {
-    "linters": {
-      "*.js": [
-        <% if (eslint === 'standard') { %>
-        "standard --fix",
-        <% } else { %>
-        "xo --fix",
-        <% } %>
-        "git add"
-      ],
-      "*.md": ["remark . -qfo", "git add"],
-      "package.json": [ "fixpack", "git add" ]
-    }
+    "*.js": [
+      "xo --fix",
+      "git add"
+    ],
+    "*.md": ["remark . -qfo", "git add"],
+    "package.json": [ "fixpack", "git add" ]
   },
   "commitlint": {
     "extends": [
@@ -91,7 +80,6 @@
     "type": "git",
     "url": "<%- repo %>"
   },
-  <% if (eslint === 'prettier') { %>
   "prettier": {
     "singleQuote": true,
     "bracketSpacing": true,
@@ -102,14 +90,9 @@
     "space": true,
     "extends": [ "xo-lass" ]
   },
-  <% } %>
   "scripts": {
     "coverage": "nyc report --reporter=text-lcov > coverage.lcov && codecov",
-    <% if (eslint === 'prettier') { %>
     "lint": "xo && remark . -qfo",
-    <% } else { %>
-    "lint": "standard && remark . -qfo",
-    <% } %>
     "test": "npm run lint && npm run ava",
     "test-coverage": "npm run lint && npm run nyc",
     "ava": "cross-env NODE_ENV=test ava",

template/test/test.js

@@ -1,7 +1,7 @@
 const test = require('ava');
 
-const Script = require('..');
 const { beforeEach, afterEach } = require('./helpers');
+const Script = require('..');
 
 test.beforeEach(beforeEach);
 test.afterEach(afterEach);

test/test.js

@@ -49,6 +49,7 @@ test('allows SPDX licenses', async t => {
       Math.floor(Math.random() * [...spdxLicenseList].length)
     ];
   };
+
   const license = getRandomLicense();
   const stream = await sao.mockPrompt(template, {
     name: 'lass',
@@ -69,12 +70,10 @@ test('allows SPDX licenses', async t => {
 // test.todo('logs error if soa fails to write license');
 
 test('defaults', async t => {
-  const stream = await sao.mockPrompt(
-    template,
-    Object.assign({}, defaults, {
-      name: 'my-package-name'
-    })
-  );
+  const stream = await sao.mockPrompt(template, {
+    ...defaults,
+    name: 'my-package-name'
+  });
 
   const ignoredFiles = ['yarn-error.log'];
   t.snapshot(
@@ -87,97 +86,63 @@ test('defaults', async t => {
 });
 
 test('username retains capital letters', async t => {
-  const stream = await sao.mockPrompt(
-    template,
-    Object.assign({}, defaults, {
-      name: 'my-package-name',
-      username: 'fooBar'
-    })
-  );
+  const stream = await sao.mockPrompt(template, {
+    ...defaults,
+    name: 'my-package-name',
+    username: 'fooBar'
+  });
   t.is(stream.meta.answers.username, 'fooBar');
   t.regex(stream.meta.answers.repo, /fooBar/);
 });
 
 test('invalid name', async t => {
   const error = await t.throwsAsync(
-    sao.mockPrompt(
-      template,
-      Object.assign({}, defaults, {
-        name: 'Foo Bar Baz Beep'
-      })
-    )
+    sao.mockPrompt(template, { ...defaults, name: 'Foo Bar Baz Beep' })
   );
   t.regex(error.message, /package name cannot have uppercase letters/);
 });
 
 test('allows scope', async t => {
   await t.notThrowsAsync(
-    sao.mockPrompt(
-      template,
-      Object.assign({}, defaults, {
-        name: '@foo/bar'
-      })
-    )
+    sao.mockPrompt(template, { ...defaults, name: '@foo/bar' })
   );
 });
 
 test('invalid version', async t => {
   const error = await t.throwsAsync(
-    sao.mockPrompt(
-      template,
-      Object.assign({}, defaults, {
-        version: 'abcdef'
-      })
-    )
+    sao.mockPrompt(template, { ...defaults, version: 'abcdef' })
   );
   t.regex(error.message, /Invalid semver version/);
 });
 
 test('invalid email', async t => {
   const error = await t.throwsAsync(
-    sao.mockPrompt(
-      template,
-      Object.assign({}, defaults, {
-        email: 'niftylettuce'
-      })
-    )
+    sao.mockPrompt(template, { ...defaults, email: 'niftylettuce' })
   );
   t.regex(error.message, /Invalid email/);
 });
 
 test('invalid website', async t => {
   const error = await t.throwsAsync(
-    sao.mockPrompt(
-      template,
-      Object.assign({}, defaults, {
-        website: 'niftylettuce'
-      })
-    )
+    sao.mockPrompt(template, { ...defaults, website: 'niftylettuce' })
   );
   t.regex(error.message, /Invalid URL/);
 });
 
 test('invalid username', async t => {
   const error = await t.throwsAsync(
-    sao.mockPrompt(
-      template,
-      Object.assign({}, defaults, {
-        username: '$$$'
-      })
-    )
+    sao.mockPrompt(template, { ...defaults, username: '$$$' })
   );
   t.regex(error.message, /Invalid GitHub username/);
 });
 
 test('invalid repo', async t => {
   const error = await t.throwsAsync(
-    sao.mockPrompt(
-      template,
-      Object.assign({}, defaults, {
-        username: 'lassjs',
-        repo: 'https://bitbucket.org/foo/bar'
-      })
-    )
+    sao.mockPrompt(template, {
+      ...defaults,
+      username: 'lassjs',
+      repo: 'https://bitbucket.org/foo/bar'
+    })
   );
   t.regex(
     error.message,