linter-bundle 2.18.0 → 2.19.0

package/CHANGELOG.md

@@ -8,6 +8,25 @@ The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/),
 
 [Show all code changes](https://github.com/jens-duttke/linter-bundle/compare/v2.18.0...HEAD)
 
+## [2.19.0] - 2022.08.28
+
+### Changed
+
+- [eslint] Updated `@typescript-eslint` from `5.33.1` to `5.35.1`
+- [eslint] Updated `eslint` from `8.22.0` to `8.23.0`
+- [eslint] Updated `eslint-import-resolver-typescript` from `3.4.2` to `3.5.0`
+- [eslint] Updated `eslint-plugin-jest` from `26.8.3` to `26.9.0`
+- [eslint] Updated `eslint-plugin-promise` from `6.0.0` to `6.0.1`
+- [eslint] Updated `eslint-plugin-react` from `7.30.1` to `7.31.1`
+- [markdownlint] Updated `markdownlint-cli` from `0.32.1` to `0.32.2`
+- [stylelint] Updated `stylelint` from `14.10.0` to `14.11.0`
+- [eslint] Activate [`allowEmptyCase`](https://github.com/eslint/eslint/blob/main/docs/src/rules/no-fallthrough.md#allowemptycase) option of `no-fallthrough` rule
+- [stylelint] Activate [`ignoreAfterCombinators: ['>', '+']`](https://stylelint.io/user-guide/rules/list/selector-max-universal/#ignoreaftercombinators-array-of-combinators) of `selector-max-universal` rule, see [this issue](https://github.com/stylelint/stylelint/issues/5792) for details
+
+### Added
+
+- [eslint/overrides-jest] Make use of [jest/prefer-each](https://github.com/jest-community/eslint-plugin-jest/blob/main/docs/rules/prefer-each.md) rule
+
 ## [2.18.0] - 2022.08.19
 
 ### Changed

package/eslint/index.js

@@ -173,7 +173,7 @@ module.exports = {
 		'no-extend-native': 'error',
 		'no-extra-bind': 'error',
 		'no-extra-label': 'error',
-		'no-fallthrough': 'error',
+		'no-fallthrough': ['error', { allowEmptyCase: true }],
 		'no-floating-decimal': 'error',
 		'no-global-assign': 'error',
 		'no-implicit-coercion': ['error', { disallowTemplateShorthand: true }],

package/eslint/overrides-jest.js

@@ -97,6 +97,7 @@ module.exports = {
 				'jest/prefer-hooks-in-order': 'error',
 				'jest/prefer-hooks-on-top': 'error',
 				'jest/prefer-comparison-matcher': 'error',
+				'jest/prefer-each': 'error',
 				'jest/prefer-equality-matcher': 'error',
 				'jest/prefer-mock-promise-shorthand': 'error',
 				'jest/prefer-snapshot-hint': 'error',

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "linter-bundle",
-	"version": "2.18.0",
+	"version": "2.19.0",
 	"description": "Ready-to use bundle of linting tools, containing configurations for ESLint, stylelint and markdownlint.",
 	"keywords": [
 		"eslint",
@@ -40,26 +40,26 @@
 		"check-outdated": "npx --yes -- check-outdated --ignore-pre-releases"
 	},
 	"dependencies": {
-		"@typescript-eslint/eslint-plugin": "5.33.1",
-		"@typescript-eslint/parser": "5.33.1",
-		"eslint": "8.22.0",
-		"eslint-import-resolver-typescript": "3.4.2",
+		"@typescript-eslint/eslint-plugin": "5.35.1",
+		"@typescript-eslint/parser": "5.35.1",
+		"eslint": "8.23.0",
+		"eslint-import-resolver-typescript": "3.5.0",
 		"eslint-import-resolver-webpack": "0.13.2",
 		"eslint-plugin-eslint-comments": "3.2.0",
 		"eslint-plugin-functional": "4.2.2",
 		"eslint-plugin-import": "2.26.0",
-		"eslint-plugin-jest": "26.8.3",
+		"eslint-plugin-jest": "26.9.0",
 		"eslint-plugin-jsdoc": "39.3.6",
 		"eslint-plugin-jsx-a11y": "6.6.1",
 		"eslint-plugin-node": "11.1.0",
-		"eslint-plugin-promise": "6.0.0",
-		"eslint-plugin-react": "7.30.1",
+		"eslint-plugin-promise": "6.0.1",
+		"eslint-plugin-react": "7.31.1",
 		"eslint-plugin-react-hooks": "4.6.0",
 		"eslint-plugin-unicorn": "43.0.2",
-		"markdownlint-cli": "0.32.1",
+		"markdownlint-cli": "0.32.2",
 		"micromatch": "4.0.5",
 		"postcss-scss": "4.0.4",
-		"stylelint": "14.10.0",
+		"stylelint": "14.11.0",
 		"stylelint-declaration-block-no-ignored-properties": "2.5.0",
 		"stylelint-order": "5.0.0",
 		"stylelint-scss": "4.3.0",
@@ -73,8 +73,8 @@
 	"devDependencies": {
 		"@types/eslint": "8.4.6",
 		"@types/micromatch": "4.0.2",
-		"@types/node": "18.7.6",
+		"@types/node": "18.7.13",
 		"stylelint-find-new-rules": "4.0.0",
-		"typescript": "4.7.4"
+		"typescript": "4.8.2"
 	}
 }

package/stylelint/index.js

@@ -290,7 +290,7 @@ module.exports = {
 			}
 		],
 		'selector-max-type': 6,
-		'selector-max-universal': 1,
+		'selector-max-universal': [1, { ignoreAfterCombinators: ['>', '+'] }],
 		'selector-nested-pattern': null,
 		'selector-no-qualifying-type': [true, { ignore: ['attribute', 'class'] }],
 		'selector-no-vendor-prefix': true,

package/stylelint/plugins/stylelint-high-performance-animation.js

@@ -122,12 +122,12 @@ const propsThatCausePaint = [
 ];
 
 /**
- * Get blacklisted properties.
+ * Get disallowed properties.
  *
  * @param {string} ignore - Property name.
- * @returns {string[]} - Array of blacklisted properties.
+ * @returns {string[]} - Array of disallowed properties.
  */
-const getBlacklist = (ignore) => {
+const getDisallowedList = (ignore) => {
 	if (ignore === 'paint-properties') {
 		return propertiesThatCauseLayout;
 	}
@@ -146,128 +146,131 @@ const getBlacklist = (ignore) => {
  */
 const unprefixed = (property) => property.replace(/^-\w+-/u, '');
 
-module.exports = stylelint.createPlugin(
-	ruleName,
-	(actual, options) => (cssRoot, result) => {
-		const validOptions = stylelint.utils.validateOptions(
-			result,
-			ruleName,
-			{ actual },
-			{
-				actual: options,
-				possible: {
-					ignore: ['paint-properties'],
-					ignoreProperties: [isString]
-				},
-				optional: true
-			}
-		);
+/**
+ * Rule function.
+ *
+ * @param {Record<string, any>} actual - Primary options
+ * @param {Record<string, any>} options - Secondary options
+ * @returns {(root: import('postcss').Root, result: import('stylelint').PostcssResult) => Promise<void> | void} PostCSS plugin
+ */
+const ruleFunction = (actual, options) => (cssRoot, result) => {
+	const validOptions = stylelint.utils.validateOptions(
+		result,
+		ruleName,
+		{ actual },
+		{
+			actual: options,
+			possible: {
+				ignore: ['paint-properties'],
+				ignoreProperties: [isString]
+			},
+			optional: true
+		}
+	);
 
-		if (!validOptions) { return; }
+	if (!validOptions) { return; }
 
-		// @ts-expect-error -- Property 'ignore' comes from an index signature, so it must be accessed with ['ignore'].
-		const blacklist = getBlacklist(options.ignore);
-		// @ts-expect-error -- Property 'ignoreProperties' comes from an index signature, so it must be accessed with ['ignoreProperties'].
-		const ignored = options.ignoreProperties ?? [];
+	const disallowedList = getDisallowedList(options['ignore']);
+	const ignored = options['ignoreProperties'] ?? [];
 
-		cssRoot.walkDecls('transition-property', (decl) => {
-			valueParser(decl.value).walk((node) => {
-				const value = unprefixed(node.value);
+	cssRoot.walkDecls('transition-property', (decl) => {
+		valueParser(decl.value).walk((node) => {
+			const value = unprefixed(node.value);
 
-				if (
-					node.type === 'word' &&
-					!ignored.includes(value) &&
-					(blacklist.includes(value) || value === 'all')
-				) {
-					const index = declarationValueIndex(decl) + node.sourceIndex;
+			if (
+				node.type === 'word' &&
+				!ignored.includes(value) &&
+				(disallowedList.includes(value) || value === 'all')
+			) {
+				const index = declarationValueIndex(decl) + node.sourceIndex;
 
-					stylelint.utils.report({
-						ruleName,
-						result,
-						node: decl,
-						message: messages.rejected('transition', node.value),
-						index
-					});
-				}
-			});
+				stylelint.utils.report({
+					ruleName,
+					result,
+					node: decl,
+					message: messages.rejected('transition', node.value),
+					index
+				});
+			}
 		});
+	});
 
-		cssRoot.walkDecls('transition', (decl) => {
-			/** @type {{ index: number; value: string; }[]} */
-			const nodes = [];
+	cssRoot.walkDecls('transition', (decl) => {
+		/** @type {{ index: number; value: string; }[]} */
+		const nodes = [];
 
-			valueParser(decl.value).walk((node) => {
-				if (node.type === 'word' || node.type === 'function') {
-					nodes.push({
-						index: node.sourceIndex,
-						value: node.value
-					});
-				}
+		valueParser(decl.value).walk((node) => {
+			if (node.type === 'word' || node.type === 'function') {
+				nodes.push({
+					index: node.sourceIndex,
+					value: node.value
+				});
+			}
 
-				return false;
-			});
+			return false;
+		});
+
+		if (!ignored.includes('all')) {
+			const transitionProp = nodes.filter((node) => {
+				const isUnit = valueParser.unit(node.value);
+				const isTimingFunction = cssTimingFunctionsRE.test(node.value);
 
-			if (!ignored.includes('all')) {
-				const transitionProp = nodes.filter((node) => {
-					const isUnit = valueParser.unit(node.value);
-					const isTimingFunction = cssTimingFunctionsRE.test(node.value);
+				if (isUnit || isTimingFunction) {
+					return false;
+				}
 
-					if (isUnit || isTimingFunction) {
-						return false;
-					}
+				return node;
+			});
 
-					return node;
+			if (nodes.length > 0 && transitionProp.length === 0) {
+				stylelint.utils.report({
+					ruleName,
+					result,
+					node: decl,
+					message: messages.rejected('transition', 'all'),
+					index: declarationValueIndex(decl) + nodes[0].index
 				});
 
-				if (nodes.length > 0 && transitionProp.length === 0) {
-					stylelint.utils.report({
-						ruleName,
-						result,
-						node: decl,
-						message: messages.rejected('transition', 'all'),
-						index: declarationValueIndex(decl) + nodes[0].index
-					});
-
-					return;
-				}
+				return;
 			}
+		}
 
-			for (const property of nodes) {
-				const index = declarationValueIndex(decl) + property.index;
-				const value = unprefixed(property.value);
+		for (const property of nodes) {
+			const index = declarationValueIndex(decl) + property.index;
+			const value = unprefixed(property.value);
 
-				if (
-					!ignored.includes(value) &&
-					(blacklist.includes(value) || value === 'all')
-				) {
-					stylelint.utils.report({
-						ruleName,
-						result,
-						node: decl,
-						message: messages.rejected('transition', property.value),
-						index
-					});
-				}
+			if (
+				!ignored.includes(value) &&
+				(disallowedList.includes(value) || value === 'all')
+			) {
+				stylelint.utils.report({
+					ruleName,
+					result,
+					node: decl,
+					message: messages.rejected('transition', property.value),
+					index
+				});
 			}
-		});
+		}
+	});
 
-		cssRoot.walkAtRules(/^keyframes$/iu, (atRuleKeyframes) => {
-			atRuleKeyframes.walkDecls((decl) => {
-				const value = unprefixed(decl.prop);
+	cssRoot.walkAtRules(/^keyframes$/iu, (atRuleKeyframes) => {
+		atRuleKeyframes.walkDecls((decl) => {
+			const value = unprefixed(decl.prop);
 
-				if (!ignored.includes(value) && blacklist.includes(value)) {
-					stylelint.utils.report({
-						ruleName,
-						result,
-						node: decl,
-						message: messages.rejected('animation', decl.prop)
-					});
-				}
-			});
+			if (!ignored.includes(value) && disallowedList.includes(value)) {
+				stylelint.utils.report({
+					ruleName,
+					result,
+					node: decl,
+					message: messages.rejected('animation', decl.prop)
+				});
+			}
 		});
-	}
-);
+	});
+};
+
+ruleFunction.ruleName = ruleName;
+ruleFunction.messages = messages;
 
-// @ts-expect-error -- Cannot assign to 'ruleName' because it is a read-only property.
-module.exports.ruleName = ruleName;
-module.exports.messages = messages;
+module.exports = stylelint.createPlugin(ruleName, ruleFunction);