eslint-config-xo 0.27.2 → 0.30.0

package/browser.js

@@ -1,10 +1,17 @@
 'use strict';
 const path = require('path');
+const confusingBrowserGlobals = require('confusing-browser-globals');
 
 module.exports = {
 	extends: path.join(__dirname, 'index.js'),
 	env: {
 		node: false,
 		browser: true
+	},
+	rules: {
+		'no-restricted-globals': [
+			'error',
+			...confusingBrowserGlobals
+		]
 	}
 };

package/esnext.js

@@ -24,8 +24,7 @@ module.exports = {
 		'prefer-numeric-literals': 'error',
 		'prefer-rest-params': 'error',
 		'prefer-spread': 'error',
-		// TODO: Enable this when targeting Node.js 8
-		// 'prefer-object-spread': 'error',
+		'prefer-object-spread': 'error',
 		'prefer-destructuring': [
 			'error',
 			{
@@ -48,8 +47,9 @@ module.exports = {
 				enforceForRenamedProperties: false
 			}
 		],
-		// TODO: Enable these when targeting Node.js 10
-		// 'no-useless-catch': 'error',
+		'no-useless-catch': 'error',
+
+		// Disabled for now as Firefox doesn't support named capture groups and I'm tired of getting issues about the use of named capture groups...
 		// 'prefer-named-capture-group': 'error'
 	}
 };

package/index.js

@@ -9,8 +9,8 @@ module.exports = {
 		}
 	},
 	env: {
-		node: true,
-		es6: true
+		es2020: true,
+		node: true
 	},
 	reportUnusedDisableDirectives: true,
 	rules: {
@@ -28,6 +28,7 @@ module.exports = {
 		'no-control-regex': 'error',
 		'no-debugger': 'error',
 		'no-dupe-args': 'error',
+		'no-dupe-else-if': 'error',
 		'no-dupe-keys': 'error',
 		'no-duplicate-case': 'error',
 		'no-empty-character-class': 'error',
@@ -59,6 +60,7 @@ module.exports = {
 		'no-obj-calls': 'error',
 		'no-prototype-builtins': 'error',
 		'no-regex-spaces': 'error',
+		'no-setter-return': 'error',
 		'no-sparse-arrays': 'error',
 		'no-template-curly-in-string': 'error',
 		'no-unreachable': 'error',
@@ -78,7 +80,12 @@ module.exports = {
 				enforceForClassMembers: true
 			}
 		],
-		'array-callback-return': 'error',
+		'array-callback-return': [
+			'error',
+			{
+				allowImplicit: true
+			}
+		],
 		'block-scoped-var': 'error',
 		complexity: 'warn',
 		curly: 'error',
@@ -90,10 +97,15 @@ module.exports = {
 			'property'
 		],
 		eqeqeq: 'error',
+		'grouped-accessor-pairs': [
+			'error',
+			'getBeforeSet'
+		],
 		'guard-for-in': 'error',
 		'no-alert': 'error',
 		'no-caller': 'error',
 		'no-case-declarations': 'error',
+		'no-constructor-return': 'error',
 		'no-else-return': [
 			'error',
 			{
@@ -191,22 +203,12 @@ module.exports = {
 				vars: 'all',
 				args: 'after-used',
 				ignoreRestSiblings: true,
-				argsIgnorePattern: '^_$',
+				argsIgnorePattern: /^_$/.source,
 				caughtErrors: 'all',
-				caughtErrorsIgnorePattern: '^_$'
+				caughtErrorsIgnorePattern: /^_$/.source
 			}
 		],
-		'handle-callback-err': 'warn',
 		'no-buffer-constructor': 'error',
-		'no-mixed-requires': [
-			'error',
-			{
-				grouping: true,
-				allowCall: true
-			}
-		],
-		'no-new-require': 'error',
-		'no-path-concat': 'error',
 		'no-restricted-imports': [
 			'error',
 			'domain',
@@ -215,14 +217,6 @@ module.exports = {
 			'sys',
 			'colors'
 		],
-		'no-restricted-modules': [
-			'error',
-			'domain',
-			'freelist',
-			'smalloc',
-			'sys',
-			'colors'
-		],
 		'array-bracket-newline': [
 			'error',
 			'consistent'
@@ -252,8 +246,8 @@ module.exports = {
 			'error',
 			'always',
 			{
-				// You can also ignore this rule by wrapping the first word in quotes
-				ignorePattern: 'pragma|ignore|prettier-ignore',
+				// You can also ignore this rule by wrapping the first word in quotes.
+				ignorePattern: /pragma|ignore|prettier-ignore/.source,
 				ignoreInlineComments: true,
 				ignoreConsecutiveComments: true
 			}
@@ -398,6 +392,7 @@ module.exports = {
 				next: '*'
 			}
 		],
+		'prefer-exponentiation-operator': 'error',
 		'quote-props': [
 			'error',
 			'as-needed'
@@ -505,7 +500,12 @@ module.exports = {
 		'no-dupe-class-members': 'error',
 		'no-new-symbol': 'error',
 		'no-this-before-super': 'error',
-		'no-useless-computed-key': 'error',
+		'no-useless-computed-key': [
+			'error',
+			{
+				enforceForClassMembers: true
+			}
+		],
 		'no-useless-constructor': 'error',
 		'no-useless-rename': 'error',
 		'require-yield': 'error',

package/license

@@ -1,6 +1,6 @@
 MIT License
 
-Copyright (c) Sindre Sorhus <sindresorhus@gmail.com> (sindresorhus.com)
+Copyright (c) Sindre Sorhus <sindresorhus@gmail.com> (https://sindresorhus.com)
 
 Permission is hereby granted, free of charge, to any person obtaining a copy of this software and associated documentation files (the "Software"), to deal in the Software without restriction, including without limitation the rights to use, copy, modify, merge, publish, distribute, sublicense, and/or sell copies of the Software, and to permit persons to whom the Software is furnished to do so, subject to the following conditions:
 

package/package.json

@@ -1,16 +1,17 @@
 {
 	"name": "eslint-config-xo",
-	"version": "0.27.2",
+	"version": "0.30.0",
 	"description": "ESLint shareable config for XO",
 	"license": "MIT",
 	"repository": "xojs/eslint-config-xo",
+	"funding": "https://github.com/sponsors/sindresorhus",
 	"author": {
 		"name": "Sindre Sorhus",
 		"email": "sindresorhus@gmail.com",
-		"url": "sindresorhus.com"
+		"url": "https://sindresorhus.com"
 	},
 	"engines": {
-		"node": ">=8"
+		"node": ">=10"
 	},
 	"scripts": {
 		"test": "ava"
@@ -49,13 +50,16 @@
 		"hint",
 		"simple"
 	],
+	"dependencies": {
+		"confusing-browser-globals": "1.0.9"
+	},
 	"devDependencies": {
 		"ava": "^2.3.0",
-		"eslint": "^6.4.0",
+		"eslint": "^7.0.0",
 		"is-plain-obj": "^2.0.0",
 		"temp-write": "^4.0.0"
 	},
 	"peerDependencies": {
-		"eslint": ">=6.4.0"
+		"eslint": ">=7"
 	}
 }

package/readme.md

@@ -14,7 +14,6 @@ See [eslint-plugin-unicorn](https://github.com/sindresorhus/eslint-plugin-unicor
 $ npm install --save-dev eslint-config-xo
 ```
 
-
 ## Usage
 
 Add some ESLint config to your `package.json`:
@@ -54,7 +53,6 @@ And [`xo/browser`](browser.js) if you're in the browser:
 }
 ```
 
-
 ## Use the XO CLI instead
 
 XO is an ESLint wrapper with great defaults.
@@ -73,14 +71,8 @@ Here are some reason why you should use the [XO CLI](https://github.com/xojs/xo)
 
 tl;dr You miss out on a lot by just using this config.
 
-
 ## Related
 
 - [eslint-config-xo-space](https://github.com/xojs/eslint-config-xo-space) - ESLint shareable config for XO with 2-space indent
 - [eslint-config-xo-react](https://github.com/xojs/eslint-config-xo-react) - ESLint shareable config for React to be used with this config
 - [eslint-config-xo-typescript](https://github.com/xojs/eslint-config-xo-typescript) - ESLint shareable config for TypeScript to be used with this config
-
-
-## License
-
-MIT © [Sindre Sorhus](https://sindresorhus.com)