Meta tweaks

Author: sindresorhus

cli.js

@@ -1,17 +1,29 @@
 #!/usr/bin/env node
 'use strict';
 const meow = require('meow');
-
 const findReadmeFile = require('./lib/find-readme-file');
 const awesomeLint = require('.');
 
+const getReporter = name => {
+	// Check if reporter is an npm package
+	try {
+		return require(name).report;
+	} catch (error) {
+		if (error.code === 'MODULE_NOT_FOUND') {
+			console.error(`No reporter found matching \`${name}\`. Using default reporter (vfile-reporter-pretty).`);
+		} else {
+			throw error;
+		}
+	}
+};
+
 const main = async () => {
 	const cli = meow(`
 		Usage
-			$ awesome-lint <optional input filename>
-		
+		  $ awesome-lint [url|filename]
+
 		Options
-			--reporter, -r Use a custom reporter
+		  --reporter, -r  Use a custom reporter
 	`, {
 		flags: {
 			reporter: {
@@ -21,27 +33,15 @@ const main = async () => {
 		}
 	});
 
-	const options = { };
 	const input = cli.input[0];
-	const reporterName = cli.flags.reporter;
 
-	if (input) {
-		options.filename = input;
-	} else {
-		options.filename = findReadmeFile(process.cwd());
-	}
+	const options = {};
+
+	options.filename = input ? input : findReadmeFile(process.cwd());
 
+	const reporterName = cli.flags.reporter;
 	if (reporterName) {
-		// Check if reporter is an npm package
-		try {
-			options.reporter = require(reporterName).report;
-		} catch (error) {
-			if (error.code === 'MODULE_NOT_FOUND') {
-				console.log(`No reporter found matching "${reporterName}". Proceeding with default reporter (vfile-reporter-pretty)`);
-			} else {
-				throw error;
-			}
-		}
+		options.reporter = getReporter(reporterName);
 	}
 
 	await awesomeLint.report(options);

package.json

@@ -104,8 +104,8 @@
 		"vfile-reporter-pretty": "^1.0.2"
 	},
 	"devDependencies": {
-		"ava": "*",
+		"ava": "^1.2.1",
 		"sinon": "^6.3.3",
-		"xo": "*"
+		"xo": "^0.24.0"
 	}
 }

test/api.js

@@ -1,8 +1,12 @@
 import test from 'ava';
-import m from '..';
+import lint from '..';
+
+// Because of https://github.com/avajs/ava/issues/2041
+// TODO: Remove this when the issue is fixed
+process.stderr.isTTY = false;
 
 test('main', async t => {
-	t.true((await m({filename: 'test/fixtures/main.md'})).messages.length > 0);
+	t.true((await lint({filename: 'test/fixtures/main.md'})).messages.length > 0);
 });
 
 test('`reporter` option', async t => {
@@ -13,6 +17,7 @@ test('`reporter` option', async t => {
 		}
 	};
 
-	await m.report({filename: 'test/fixtures/main.md', reporter});
+	await lint.report({filename: 'test/fixtures/main.md', reporter});
+
 	t.true(wasReporterCalled);
 });

test/cli.js

@@ -2,6 +2,8 @@ import test from 'ava';
 import execa from 'execa';
 
 test('main', async t => {
-	const error = await t.throws(execa.stdout('./cli.js', ['test/fixtures/main.md']));
-	t.regex(error.message, /Missing Awesome badge/);
+	await t.throwsAsync(
+		execa.stdout('./cli.js', ['test/fixtures/main.md']),
+		/Missing Awesome badge/
+	);
 });