Revert "Add option to no-extra-boolean-cast"

index.js

@@ -40,12 +40,14 @@
 		],
 		'no-empty-static-block': 'error',
 		'no-ex-assign': 'error',
-		'no-extra-boolean-cast': [
-			'error',
-			{
-				enforceForInnerExpressions: true
-			},
-		],
+		'no-extra-boolean-cast': 'error',
+		// Disabled because of https://github.com/xojs/eslint-config-xo/pull/87
+		// 'no-extra-boolean-cast': [
+		// 	'error',
+		// 	{
+		// 		enforceForInnerExpressions: true
+		// 	},
+		// ],
 		// Disabled because of https://github.com/eslint/eslint/issues/6028
 		// 'no-extra-parens': [
 		// 	'error',
@@ -209,7 +211,7 @@
 		radix: 'error',
 
 		// Disabled for now as it causes too much churn
 		// TODO: Enable it in the future when I have time to deal with
 		// the churn and the rule is stable and has an autofixer.
 		// Still doesn't have a fixer as of ESLint 7.24.0.
 		// 'require-unicode-regexp': 'error',
@@ -227,7 +229,7 @@
 		'no-restricted-globals': [
 			'error',
 			'event',
 			// TODO: Enable this in 2025.
 			// {
 			// 	name: 'Buffer',
 			// 	message: 'Use Uint8Array instead. See: https://sindresorhus.com/blog/goodbye-nodejs-buffer',
@@ -271,7 +273,7 @@
 			'sys',
 			'querystring',
 			'colors',
 			// TODO: Enable this in 2025.
 			// {
 			// 	name: 'buffer',
 			// 	message: 'Use Uint8Array instead. See: https://sindresorhus.com/blog/goodbye-nodejs-buffer',
@@ -642,7 +644,7 @@
 			{
 				// `array` is disabled because it forces destructuring on
 				// stupid stuff like `foo.bar = process.argv[2];`
 				// TODO: Open ESLint issue about this
 				VariableDeclarator: {
 					array: false,
 					object: true,