chore(package): update xo to version 0.23.0

Author: greenkeeper[bot]

index.js

@@ -4,7 +4,7 @@ const prepareGit = require('./lib/prepare');
 
 let verified;
 
-async function verifyConditions(pluginConfig, context) {
+function verifyConditions(pluginConfig, context) {
   const {options} = context;
   // If the Git prepare plugin is used and has `assets` or `message` configured, validate them now in order to prevent any release if the configuration is wrong
   if (options.prepare) {
@@ -14,13 +14,13 @@ async function verifyConditions(pluginConfig, context) {
     pluginConfig.assets = defaultTo(pluginConfig.assets, preparePlugin.assets);
     pluginConfig.message = defaultTo(pluginConfig.message, preparePlugin.message);
   }
-  await verifyGit(pluginConfig);
+  verifyGit(pluginConfig);
   verified = true;
 }
 
 async function prepare(pluginConfig, context) {
   if (!verified) {
-    await verifyGit(pluginConfig);
+    verifyGit(pluginConfig);
     verified = true;
   }
   await prepareGit(pluginConfig, context);

lib/git.js

@@ -59,7 +59,7 @@ async function push(origin, branch, execaOpts) {
  *
  * @return {String} The sha of the head commit on the local repository
  */
-async function gitHead(execaOpts) {
+function gitHead(execaOpts) {
   return execa.stdout('git', ['rev-parse', 'HEAD'], execaOpts);
 }
 

lib/verify.js

@@ -23,7 +23,7 @@ const VALIDATORS = {
  * @param {String|Array<String|Object>} [pluginConfig.assets] Files to include in the release commit. Can be files path or globs.
  * @param {String} [pluginConfig.message] The commit message for the release.
  */
-module.exports = async pluginConfig => {
+module.exports = pluginConfig => {
   const options = resolveConfig(pluginConfig);
 
   const errors = Object.entries(options).reduce(

package.json

@@ -40,7 +40,7 @@
     "semantic-release": "^15.0.0",
     "sinon": "^6.0.0",
     "tempy": "^0.2.1",
-    "xo": "^0.22.0"
+    "xo": "^0.23.0"
   },
   "engines": {
     "node": ">=8.3"

test/helpers/git-utils.js

@@ -60,7 +60,7 @@ export async function initBareRepo(repositoryUrl, branch = 'master') {
  * @returns {Array<Commit>} The created commits, in reverse order (to match `git log` order).
  */
 export async function gitCommits(messages, execaOpts) {
-  await pReduce(messages, async (_, message) =>
+  await pReduce(messages, (_, message) =>
     execa.stdout('git', ['commit', '-m', message, '--allow-empty', '--no-gpg-sign'], execaOpts)
   );
   return (await gitGetCommits(undefined, execaOpts)).slice(0, messages.length);

test/integration.test.js

@@ -97,29 +97,29 @@ test('Verify authentication only on the fist call', async t => {
   const nextRelease = {version: '2.0.0', gitTag: 'v2.0.0'};
   const options = {repositoryUrl, branch, prepare: ['@semantic-release/npm']};
 
-  await t.notThrows(t.context.m.verifyConditions({}, {cwd, options, logger: t.context.logger}));
+  t.notThrows(() => t.context.m.verifyConditions({}, {cwd, options, logger: t.context.logger}));
   await t.context.m.prepare({}, {cwd, options: {repositoryUrl, branch}, nextRelease, logger: t.context.logger});
 });
 
-test('Throw SemanticReleaseError if prepare config is invalid', async t => {
+test('Throw SemanticReleaseError if prepare config is invalid', t => {
   const message = 42;
   const assets = true;
   const options = {prepare: ['@semantic-release/npm', {path: '@semantic-release/git', message, assets}]};
 
-  const errors = [...(await t.throws(t.context.m.verifyConditions({}, {options, logger: t.context.logger})))];
+  const errors = [...t.throws(() => t.context.m.verifyConditions({}, {options, logger: t.context.logger}))];
 
   t.is(errors[0].name, 'SemanticReleaseError');
   t.is(errors[0].code, 'EINVALIDASSETS');
   t.is(errors[1].name, 'SemanticReleaseError');
   t.is(errors[1].code, 'EINVALIDMESSAGE');
 });
 
-test('Throw SemanticReleaseError if config is invalid', async t => {
+test('Throw SemanticReleaseError if config is invalid', t => {
   const message = 42;
   const assets = true;
 
   const errors = [
-    ...(await t.throws(t.context.m.verifyConditions({message, assets}, {options: {}, logger: t.context.logger}))),
+    ...t.throws(() => t.context.m.verifyConditions({message, assets}, {options: {}, logger: t.context.logger})),
   ];
 
   t.is(errors[0].name, 'SemanticReleaseError');

test/verify.test.js

@@ -1,88 +1,88 @@
 import test from 'ava';
 import verify from '../lib/verify';
 
-test('Throw SemanticReleaseError if "assets" option is not a String or false or an Array of Objects', async t => {
+test('Throw SemanticReleaseError if "assets" option is not a String or false or an Array of Objects', t => {
   const assets = true;
-  const [error] = await t.throws(verify({assets}));
+  const [error] = t.throws(() => verify({assets}));
 
   t.is(error.name, 'SemanticReleaseError');
   t.is(error.code, 'EINVALIDASSETS');
 });
 
-test('Throw SemanticReleaseError if "assets" option is not an Array with invalid elements', async t => {
+test('Throw SemanticReleaseError if "assets" option is not an Array with invalid elements', t => {
   const assets = ['file.js', 42];
-  const [error] = await t.throws(verify({assets}));
+  const [error] = t.throws(() => verify({assets}));
 
   t.is(error.name, 'SemanticReleaseError');
   t.is(error.code, 'EINVALIDASSETS');
 });
 
-test('Verify "assets" is a String', async t => {
+test('Verify "assets" is a String', t => {
   const assets = 'file2.js';
 
-  await t.notThrows(verify({assets}));
+  t.notThrows(() => verify({assets}));
 });
 
-test('Verify "assets" is an Array of String', async t => {
+test('Verify "assets" is an Array of String', t => {
   const assets = ['file1.js', 'file2.js'];
 
-  await t.notThrows(verify({assets}));
+  t.notThrows(() => verify({assets}));
 });
 
-test('Verify "assets" is an Object with a path property', async t => {
+test('Verify "assets" is an Object with a path property', t => {
   const assets = {path: 'file2.js'};
 
-  await t.notThrows(verify({assets}));
+  t.notThrows(() => verify({assets}));
 });
 
-test('Verify "assets" is an Array of Object with a path property', async t => {
+test('Verify "assets" is an Array of Object with a path property', t => {
   const assets = [{path: 'file1.js'}, {path: 'file2.js'}];
 
-  await t.notThrows(verify({assets}));
+  t.notThrows(() => verify({assets}));
 });
 
-test('Verify disabled "assets" (set to false)', async t => {
+test('Verify disabled "assets" (set to false)', t => {
   const assets = false;
 
-  await t.notThrows(verify({assets}));
+  t.notThrows(() => verify({assets}));
 });
 
-test('Verify "assets" is an Array of glob Arrays', async t => {
+test('Verify "assets" is an Array of glob Arrays', t => {
   const assets = [['dist/**', '!**/*.js'], 'file2.js'];
 
-  await t.notThrows(verify({assets}));
+  t.notThrows(() => verify({assets}));
 });
 
-test('Verify "assets" is an Array of Object with a glob Arrays in path property', async t => {
+test('Verify "assets" is an Array of Object with a glob Arrays in path property', t => {
   const assets = [{path: ['dist/**', '!**/*.js']}, {path: 'file2.js'}];
 
-  await t.notThrows(verify({assets}));
+  t.notThrows(() => verify({assets}));
 });
 
-test('Throw SemanticReleaseError if "message" option is not a String', async t => {
+test('Throw SemanticReleaseError if "message" option is not a String', t => {
   const message = 42;
-  const [error] = await t.throws(verify({message}));
+  const [error] = t.throws(() => verify({message}));
 
   t.is(error.name, 'SemanticReleaseError');
   t.is(error.code, 'EINVALIDMESSAGE');
 });
 
-test('Throw SemanticReleaseError if "message" option is an empty String', async t => {
+test('Throw SemanticReleaseError if "message" option is an empty String', t => {
   const message = '';
-  const [error] = await t.throws(verify({message}));
+  const [error] = t.throws(() => verify({message}));
 
   t.is(error.name, 'SemanticReleaseError');
   t.is(error.code, 'EINVALIDMESSAGE');
 });
 
-test('Throw SemanticReleaseError if "message" option is a whitespace String', async t => {
+test('Throw SemanticReleaseError if "message" option is a whitespace String', t => {
   const message = '  \n \r ';
-  const [error] = await t.throws(verify({message}));
+  const [error] = t.throws(() => verify({message}));
 
   t.is(error.name, 'SemanticReleaseError');
   t.is(error.code, 'EINVALIDMESSAGE');
 });
 
-test('Verify undefined "message" and "assets"', async t => {
-  await t.notThrows(verify({}));
+test('Verify undefined "message" and "assets"', t => {
+  t.notThrows(() => verify({}));
 });