linter-bundle 3.4.0 → 3.5.0

package/CHANGELOG.md

@@ -6,7 +6,28 @@ The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/),
 
 ## [Unreleased]
 
-[Show all code changes](https://github.com/jens-duttke/linter-bundle/compare/v3.4.0...HEAD)
+[Show all code changes](https://github.com/jens-duttke/linter-bundle/compare/v3.5.0...HEAD)
+
+## [3.5.0] - 2023-07-28
+
+### Changed
+
+- [eslint] Updated `@typescript-eslint/eslint-plugin` from `6.1.0` to `6.2.0`
+- [eslint] Updated `eslint-plugin-import` from `2.27.5` to `2.28.0`
+- [eslint] Updated `eslint-plugin-jsdoc` from `46.4.4` to `46.4.5`
+- [eslint] Updated `eslint-plugin-react` from `7.32.2` to `7.33.0`
+- [eslint] Updated `eslint-plugin-unicorn` from `48.0.0` to `48.0.1`
+- [eslint] Reactivated `unicorn/no-empty-file`
+- [eslint] Activated `ignoreConditionalTests` and `ignoreMixedLogicalExpressions` options of [@typescript-eslint/prefer-nullish-coalescing](https://github.com/typescript-eslint/typescript-eslint/blob/main/packages/eslint-plugin/docs/rules/prefer-nullish-coalescing.md) rule
+- [styleint] Set `ignoreFunctions` option to `/^custom-/u` of [`scss/function-no-unknown`](https://github.com/stylelint-scss/stylelint-scss/blob/master/src/rules/function-no-unknown/README.md) rule
+- [stylelint] Ignore SCSS variables in [`declaration-property-value-no-unknown`](https://stylelint.io/user-guide/rules/declaration-property-value-no-unknown/) rule
+- [stylelint] Ignore wrapping parentheses for `grid-template-areas` values in [`declaration-property-value-no-unknown`](https://stylelint.io/user-guide/rules/declaration-property-value-no-unknown/) rule
+
+### Added
+
+- Replace the ESLint rule `class-methods-use-this` by the new [`@typescript-eslint/class-methods-use-this`](https://typescript-eslint.io/rules/class-methods-use-this/) rule
+
+[Show all code changes](https://github.com/jens-duttke/linter-bundle/compare/v3.4.0...v3.5.0)
 
 ## [3.4.0] - 2023-07-20
 

package/eslint/index.js

@@ -125,7 +125,7 @@ module.exports = {
 				'brace-style': 'off', // Covered by @typescript-eslint/brace-style
 				'camelcase': 'off', // Covered by @typescript-eslint/naming-convention
 				'capitalized-comments': 'off',
-				'class-methods-use-this': ['error', { exceptMethods: ['componentDidMount', 'componentDidUpdate', 'componentWillUnmount', 'shouldComponentUpdate'] }],
+				'class-methods-use-this': 'off', // Covered by @typescript-eslint/class-methods-use-this
 				'comma-dangle': 'off', // Covered by @typescript-eslint/comma-dangle
 				'comma-spacing': 'off', // Covered by @typescript-eslint/comma-spacing
 				'comma-style': 'error',
@@ -423,9 +423,8 @@ module.exports = {
 				}],
 				'@typescript-eslint/brace-style': ['error', 'stroustrup', { allowSingleLine: true }],
 				'@typescript-eslint/class-literal-property-style': 'error',
-				'@typescript-eslint/comma-dangle': ['error', {
-					generics: 'ignore'
-				}],
+				'@typescript-eslint/class-methods-use-this': ['error', { exceptMethods: ['componentDidMount', 'componentDidUpdate', 'componentWillUnmount', 'shouldComponentUpdate'] }],
+				'@typescript-eslint/comma-dangle': ['error', { generics: 'ignore' }],
 				'@typescript-eslint/comma-spacing': 'error',
 				'@typescript-eslint/consistent-generic-constructors': 'error',
 				'@typescript-eslint/consistent-indexed-object-style': 'error',
@@ -745,7 +744,7 @@ module.exports = {
 				'@typescript-eslint/prefer-includes': 'error',
 				'@typescript-eslint/prefer-literal-enum-member': 'error',
 				'@typescript-eslint/prefer-namespace-keyword': 'error',
-				'@typescript-eslint/prefer-nullish-coalescing': 'error',
+				'@typescript-eslint/prefer-nullish-coalescing': ['error', { ignoreConditionalTests: true, ignoreMixedLogicalExpressions: true }],
 				'@typescript-eslint/prefer-optional-chain': 'error',
 				'@typescript-eslint/prefer-readonly': 'error',
 				'@typescript-eslint/prefer-readonly-parameter-types': ['off', { ignoreInferredTypes: true }], // @todo An interface should ensure that it's properties are readonly, not every usage of it
@@ -961,7 +960,7 @@ module.exports = {
 				'unicorn/no-await-expression-member': 'error',
 				'unicorn/no-console-spaces': 'error',
 				'unicorn/no-document-cookie': 'error',
-				'unicorn/no-empty-file': 'off', // @todo False-positives since @typescript-eslint v6.0.0. @see https://github.com/sindresorhus/eslint-plugin-unicorn/issues/2175
+				'unicorn/no-empty-file': 'error',
 				'unicorn/no-for-loop': 'off', // @typescript-eslint/prefer-for-of
 				'unicorn/no-hex-escape': 'error',
 				'unicorn/no-instanceof-array': 'error',

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "linter-bundle",
-	"version": "3.4.0",
+	"version": "3.5.0",
 	"description": "Ready-to use bundle of linting tools, containing configurations for ESLint, stylelint and markdownlint.",
 	"keywords": [
 		"eslint",
@@ -40,23 +40,23 @@
 		"check-outdated": "npx --yes -- check-outdated --ignore-pre-releases"
 	},
 	"dependencies": {
-		"@typescript-eslint/eslint-plugin": "6.1.0",
-		"@typescript-eslint/parser": "6.1.0",
-		"@typescript-eslint/utils": "6.1.0",
+		"@typescript-eslint/eslint-plugin": "6.2.0",
+		"@typescript-eslint/parser": "6.2.0",
+		"@typescript-eslint/utils": "6.2.0",
 		"eslint": "8.45.0",
 		"eslint-import-resolver-typescript": "3.5.5",
 		"eslint-import-resolver-webpack": "0.13.2",
 		"eslint-plugin-eslint-comments": "3.2.0",
 		"eslint-plugin-functional": "5.0.8",
-		"eslint-plugin-import": "2.27.5",
+		"eslint-plugin-import": "2.28.0",
 		"eslint-plugin-jest": "27.2.3",
-		"eslint-plugin-jsdoc": "46.4.4",
+		"eslint-plugin-jsdoc": "46.4.5",
 		"eslint-plugin-jsx-a11y": "6.7.1",
 		"eslint-plugin-n": "16.0.1",
 		"eslint-plugin-promise": "6.1.1",
-		"eslint-plugin-react": "7.32.2",
+		"eslint-plugin-react": "7.33.0",
 		"eslint-plugin-react-hooks": "4.6.0",
-		"eslint-plugin-unicorn": "48.0.0",
+		"eslint-plugin-unicorn": "48.0.1",
 		"markdownlint-cli": "0.35.0",
 		"micromatch": "4.0.5",
 		"postcss-scss": "4.0.6",
@@ -71,9 +71,9 @@
 		"typescript": "^4.0.0"
 	},
 	"devDependencies": {
-		"@types/eslint": "8.44.0",
+		"@types/eslint": "8.44.1",
 		"@types/micromatch": "4.0.2",
-		"@types/node": "20.4.2",
+		"@types/node": "20.4.5",
 		"stylelint-find-new-rules": "4.1.0",
 		"typescript": "5.1.6"
 	}

package/stylelint/index.js

@@ -155,7 +155,9 @@ module.exports = {
 			true,
 			{
 				ignoreProperties: {
-					'initial-value': '/.+/'
+					'initial-value': /.+/u,
+					'grid-template-areas': /^\([^\u0000]+\)$/u, // Whole block wrapped in parentheses.
+					'/.+/': /((^|\s)\$[a-z])|/ui // SCSS variables
 				}
 			}
 		],
@@ -901,7 +903,8 @@ module.exports = {
 		'scss/dollar-variable-empty-line-after': [
 			'always',
 			{
-				except: ['last-nested', 'before-dollar-variable']
+				except: ['last-nested', 'before-dollar-variable'],
+				ignore: ['before-comment', 'inside-single-line-block']
 			}
 		],
 		'scss/dollar-variable-first-in-block': [true, { ignore: ['comments', 'imports'] }],
@@ -918,7 +921,7 @@ module.exports = {
 		'scss/double-slash-comment-inline': null,
 		'scss/double-slash-comment-whitespace-inside': 'always',
 		'scss/function-color-relative': true,
-		'scss/function-no-unknown': true,
+		'scss/function-no-unknown': [true, { ignoreFunctions: [/^custom-/u] }],
 		'scss/function-quote-no-quoted-strings-inside': true,
 		'scss/function-unquote-no-unquoted-strings-inside': true,
 		'scss/map-keys-quotes': 'always',