xo 0.35.0 → 0.37.1

package/cli-main.js

@@ -135,6 +135,10 @@ if (typeof options.space === 'string') {
 	}
 }
 
+if (process.env.GITHUB_ACTIONS && !options.fix && !options.reporter) {
+	options.quiet = true;
+}
+
 const log = report => {
 	const reporter = options.reporter || process.env.GITHUB_ACTIONS ? xo.getFormatter(options.reporter || 'compact') : formatterPretty;
 	process.stdout.write(reporter(report.results));
@@ -171,7 +175,7 @@ if (options.nodeVersion) {
 		if (options.fix) {
 			const result = xo.lintText(stdin, options).results[0];
 			// If there is no output, pass the stdin back out
-			process.stdout.write(result.output || stdin);
+			process.stdout.write((result && result.output) || stdin);
 			return;
 		}
 

package/config/plugins.js

@@ -42,6 +42,20 @@ module.exports = {
 				checkShorthandImports: false,
 				extendDefaultReplacements: false,
 				replacements: {
+					// https://thenextweb.com/dd/2020/07/13/linux-kernel-will-no-longer-use-terms-blacklist-and-slave/
+					whitelist: {
+						allowList: true
+					},
+					blacklist: {
+						denyList: true
+					},
+					master: {
+						main: true
+					},
+					slave: {
+						secondary: true
+					},
+
 					// Not part of `eslint-plugin-unicorn`
 					application: {
 						app: true
@@ -109,6 +123,9 @@ module.exports = {
 					},
 					val: {
 						value: true
+					},
+					err: {
+						error: true
 					}
 				}
 			}
@@ -135,6 +152,9 @@ module.exports = {
 			}
 		],
 
+		// TODO: Remove this when https://github.com/sindresorhus/eslint-plugin-unicorn/pull/977 is out.
+		'unicorn/import-index': 'off',
+
 		// TODO: Disabled for now as I don't have time to deal with the backslash that might come from this. Try to enable this rule in 2021.
 		'unicorn/no-null': 'off',
 
@@ -162,20 +182,22 @@ module.exports = {
 		'import/export': 'error',
 		'import/extensions': [
 			'error',
+			'always',
 			{
-				js: 'never',
-				jsx: 'never',
-				json: 'always',
-				svg: 'always',
-				css: 'always'
+				ignorePackages: true,
+				// TypeScript doesn't yet support using extensions and fails with error TS2691.
+				pattern: {
+					ts: 'never',
+					tsx: 'never'
+				}
 			}
 		],
 
-		// Disabled as it causes problems with TypeScript when you use mixed ESM and CommonJS
-		// TODO: Enable again when I target only ESM
+		// Disabled as it causes problems with TypeScript when you use mixed ESM and CommonJS.
+		// TODO: Enable again when I target only ESM.
 		// 'import/first': 'error',
 
-		// Disabled as it doesn't work with TypeScript
+		// Disabled as it doesn't work with TypeScript.
 		// This issue and some others: https://github.com/benmosher/eslint-plugin-import/issues/1341
 		// 'import/named': 'error',
 
@@ -194,12 +216,7 @@ module.exports = {
 		// Enable this sometime in the future when Node.js has ES2015 module support
 		// 'import/no-cycle': 'error'
 
-		'import/no-useless-path-segments': [
-			'error',
-			{
-				noUselessIndex: true
-			}
-		],
+		'import/no-useless-path-segments': 'error',
 
 		// Disabled as it doesn't work with TypeScript
 		// 'import/newline-after-import': 'error',
@@ -225,7 +242,7 @@ module.exports = {
 		// 'import/no-unresolved': [
 		// 	'error',
 		// 	{
-		// 		commonjs: true
+		// 		commonjs: false
 		// 	}
 		// ],
 
@@ -247,11 +264,11 @@ module.exports = {
 			}
 		],
 
-		// Redundant with import/no-extraneous-dependencies
+		// Redundant with `import/no-extraneous-dependencies`.
 		// 'node/no-extraneous-import': 'error',
 		// 'node/no-extraneous-require': 'error',
 
-		// Redundant with import/no-unresolved
+		// Redundant with `import/no-unresolved`.
 		// 'node/no-missing-import': 'error',
 		// 'node/no-missing-require': 'error',
 
@@ -340,7 +357,7 @@ module.exports = {
 		'eslint-comments/no-aggregating-enable': 'error',
 		'eslint-comments/no-duplicate-disable': 'error',
 
-		// Disabled as it's already covered by the `unicorn/no-abusive-eslint-disable` rule
+		// Disabled as it's already covered by the `unicorn/no-abusive-eslint-disable` rule.
 		// 'eslint-comments/no-unlimited-disable': 'error',
 
 		'eslint-comments/no-unused-disable': 'error',

package/lib/options-manager.js

@@ -327,6 +327,10 @@ const buildXOConfig = options => config => {
 		config.baseConfig.parser = options.parser;
 	}
 
+	if (options.processor) {
+		config.baseConfig.processor = options.processor;
+	}
+
 	config.baseConfig.settings = options.settings || {};
 	config.baseConfig.settings['import/resolver'] = gatherImportResolvers(options);
 

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "xo",
-	"version": "0.35.0",
+	"version": "0.37.1",
 	"description": "JavaScript/TypeScript linter with great defaults",
 	"license": "MIT",
 	"repository": "xojs/xo",
@@ -51,15 +51,15 @@
 		"typescript"
 	],
 	"dependencies": {
-		"@typescript-eslint/eslint-plugin": "^4.8.1",
-		"@typescript-eslint/parser": "^4.8.1",
+		"@typescript-eslint/eslint-plugin": "^4.11.1",
+		"@typescript-eslint/parser": "^4.11.1",
 		"arrify": "^2.0.1",
 		"cosmiconfig": "^7.0.0",
 		"debug": "^4.3.1",
-		"eslint": "^7.13.0",
-		"eslint-config-prettier": "^6.15.0",
-		"eslint-config-xo": "^0.33.1",
-		"eslint-config-xo-typescript": "^0.36.0",
+		"eslint": "^7.17.0",
+		"eslint-config-prettier": "^7.1.0",
+		"eslint-config-xo": "^0.34.0",
+		"eslint-config-xo-typescript": "^0.37.0",
 		"eslint-formatter-pretty": "^4.0.0",
 		"eslint-import-resolver-webpack": "^0.13.0",
 		"eslint-plugin-ava": "^11.0.0",
@@ -67,50 +67,52 @@
 		"eslint-plugin-import": "^2.22.1",
 		"eslint-plugin-no-use-extend-native": "^0.5.0",
 		"eslint-plugin-node": "^11.1.0",
-		"eslint-plugin-prettier": "^3.1.4",
+		"eslint-plugin-prettier": "^3.3.0",
 		"eslint-plugin-promise": "^4.2.1",
-		"eslint-plugin-unicorn": "^23.0.0",
+		"eslint-plugin-unicorn": "^25.0.1",
 		"find-cache-dir": "^3.3.1",
 		"find-up": "^5.0.0",
 		"fs-extra": "^9.0.1",
 		"get-stdin": "^8.0.0",
-		"globby": "^9.0.0",
+		"globby": "^9.2.0",
 		"has-flag": "^4.0.0",
 		"imurmurhash": "^0.1.4",
 		"is-path-inside": "^3.0.2",
 		"json-stable-stringify-without-jsonify": "^1.0.1",
 		"json5": "^2.1.3",
 		"lodash": "^4.17.20",
-		"meow": "^8.0.0",
+		"meow": "^8.1.0",
 		"micromatch": "^4.0.2",
-		"open-editor": "^2.0.1",
+		"open-editor": "^3.0.0",
 		"p-reduce": "^2.1.0",
 		"path-exists": "^4.0.0",
-		"prettier": "^2.1.2",
+		"prettier": "^2.2.1",
 		"resolve-cwd": "^3.0.0",
 		"resolve-from": "^5.0.0",
-		"semver": "^7.3.2",
+		"semver": "^7.3.4",
 		"slash": "^3.0.0",
 		"to-absolute-glob": "^2.0.2",
-		"typescript": "^4.1.2",
+		"typescript": "^4.1.3",
 		"update-notifier": "^5.0.1"
 	},
 	"devDependencies": {
-		"ava": "^3.13.0",
-		"coveralls": "^3.1.0",
+		"ava": "^3.15.0",
 		"eslint-config-xo-react": "^0.23.0",
-		"eslint-plugin-react": "^7.21.5",
+		"eslint-plugin-react": "^7.22.0",
 		"eslint-plugin-react-hooks": "^4.2.0",
 		"esm": "^3.2.25",
-		"execa": "^4.1.0",
+		"execa": "^5.0.0",
 		"nyc": "^15.1.0",
 		"pify": "^5.0.0",
 		"proxyquire": "^2.1.3",
 		"temp-write": "^4.0.0",
-		"webpack": "^5.6.0"
+		"webpack": "^5.11.1"
 	},
 	"eslintConfig": {
-		"extends": "eslint-config-xo"
+		"extends": "eslint-config-xo",
+		"rules": {
+			"unicorn/import-index": "off"
+		}
 	},
 	"eslintIgnore": [
 		"test/fixtures"
@@ -120,5 +122,11 @@
 			"esm"
 		],
 		"timeout": "1m"
+	},
+	"nyc": {
+		"reporter": [
+			"text",
+			"lcov"
+		]
 	}
 }

package/readme.md

@@ -8,7 +8,8 @@
 
 > JavaScript/TypeScript linter with great defaults
 
-[![Build Status](https://travis-ci.com/xojs/xo.svg?branch=master)](https://travis-ci.com/xojs/xo) [![Coverage Status](https://coveralls.io/repos/github/xojs/xo/badge.svg?branch=master)](https://coveralls.io/github/xojs/xo?branch=master) [![XO code style](https://img.shields.io/badge/code_style-XO-5ed9c7.svg)](https://github.com/xojs/xo) [![Gitter](https://badges.gitter.im/join_chat.svg)](https://gitter.im/xojs/Lobby)
+[![Coverage Status](https://codecov.io/gh/xojs/xo/branch/master/graph/badge.svg)](https://codecov.io/gh/xojs/xo/branch/master)
+[![XO code style](https://img.shields.io/badge/code_style-XO-5ed9c7.svg)](https://github.com/xojs/xo)
 
 Opinionated but configurable ESLint wrapper with lots of goodies included. Enforces strict and readable code. Never discuss code style on a pull request again! No decision-making. No `.eslintrc` or `.jshintrc` to manage. It just works!
 
@@ -260,6 +261,12 @@ Type: `string`
 
 ESLint parser. For example, [`babel-eslint`](https://github.com/babel/babel-eslint) if you're using language features that ESLint doesn't yet support.
 
+### processor
+
+Type: `string`
+
+[ESLint processor.](https://eslint.org/docs/user-guide/configuring#specifying-processor)
+
 ### esnext
 
 Type: `boolean`\
@@ -292,6 +299,10 @@ XO will handle the [@typescript-eslint/parser `project` option](https://github.c
 
 See [eslint-config-xo-flow#use-with-xo](https://github.com/xojs/eslint-config-xo-flow#use-with-xo)
 
+## GitHub Actions
+
+XO uses a different formatter when running in a GitHub Actions workflow to be able to get [inline annotations](https://developer.github.com/changes/2019-09-06-more-check-annotations-shown-in-files-changed-tab/). XO also disables warnings here.
+
 ## Config Overrides
 
 XO makes it easy to override configs for specific files. The `overrides` property must be an array of override objects. Each override object must contain a `files` property which is a glob string, or an array of glob strings, relative to the config file. The remaining properties are identical to those described above, and will override the settings of the base config. If multiple override configs match the same file, each matching override is applied in the order it appears in the array. This means the last override in the array takes precedence over earlier ones. Consider the following example: