Require Node.js 12 and ESLint 8

Author: sindresorhus

Diff for: .github/workflows/main.yml

@@ -10,12 +10,12 @@ jobs:
       fail-fast: false
       matrix:
         node-version:
+          - 16
           - 14
           - 12
-          - 10
     steps:
       - uses: actions/checkout@v2
-      - uses: actions/setup-node@v1
+      - uses: actions/setup-node@v2
         with:
           node-version: ${{ matrix.node-version }}
       - run: npm install

Diff for: package.json

@@ -11,7 +11,7 @@
 		"url": "https://sindresorhus.com"
 	},
 	"engines": {
-		"node": ">=10"
+		"node": ">=12"
 	},
 	"scripts": {
 		"test": "ava"
@@ -52,10 +52,10 @@
 	},
 	"devDependencies": {
 		"ava": "^2.4.0",
-		"eslint": "^7.20.0",
+		"eslint": "^8.6.0",
 		"is-plain-obj": "^3.0.0"
 	},
 	"peerDependencies": {
-		"eslint": ">=7.32.0"
+		"eslint": ">=8.6.0"
 	}
 }

Diff for: readme.md

@@ -8,8 +8,8 @@ This is for advanced users. [You probably want to use XO directly.](https://gith
 
 ## Install
 
-```
-$ npm install --save-dev eslint-config-xo-space
+```sh
+npm install --save-dev eslint-config-xo-space
 ```
 
 ## Usage

Diff for: test/test.js

@@ -1,29 +1,38 @@
 import test from 'ava';
 import isPlainObj from 'is-plain-obj';
-import eslint from 'eslint';
-import path from 'path';
+import {ESLint} from 'eslint';
 
-const fixture = `'use strict';\nconst x = true;\n\nif (x) {\n  console.log();\n}\n`;
+const hasRule = (errors, ruleId) => errors.some(error => error.ruleId === ruleId);
 
-function runEslint(str, conf) {
-	const linter = new eslint.CLIEngine({
+const fixture = `"use strict";\nconst x = true;\n\nif (x) {\n  console.log();\n}\n`;
+
+async function runEslint(string, config) {
+	const eslint = new ESLint({
 		useEslintrc: false,
-		configFile: path.join(__dirname, conf)
+		overrideConfig: config,
 	});
 
-	return linter.executeOnText(str).results[0].messages;
+	const [firstResult] = await eslint.lintText(string);
+
+	return firstResult.messages;
 }
 
-test('main', t => {
-	const conf = require('../');
-	t.true(isPlainObj(conf));
-	t.true(isPlainObj(conf.rules));
-	t.is(runEslint(fixture, '../index.js').length, 0);
+test('main', async t => {
+	const config = require('../index.js');
+
+	t.true(isPlainObj(config));
+	t.true(isPlainObj(config.rules));
+
+	const errors = await runEslint(fixture, config);
+	t.true(hasRule(errors, 'quotes'), JSON.stringify(errors));
 });
 
-test('browser', t => {
-	const conf = require('../browser');
-	t.true(isPlainObj(conf));
-	t.true(isPlainObj(conf.rules));
-	t.is(runEslint(fixture, '../browser.js').length, 0);
+test('browser', async t => {
+	const config = require('../browser.js');
+
+	t.true(isPlainObj(config));
+	t.true(isPlainObj(config.rules));
+
+	const errors = await runEslint(fixture, config);
+	t.true(hasRule(errors, 'quotes'), JSON.stringify(errors));
 });