linter-bundle 3.4.0 → 3.6.0

package/CHANGELOG.md

@@ -6,7 +6,44 @@ 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.6.0...HEAD)
+
+## [3.6.0] - 2023-08-11
+
+- [general] Change peer-dependency of `typescript` from `^4.0.0` to `>=4.0.0`
+- [eslint] Updated `@typescript-eslint` from `6.2.0` to `6.3.0`
+- [eslint] Updated `eslint` from `8.45.0` to `8.47.0`
+- [eslint] Updated `eslint-import-resolver-typescript` from `3.5.5` to `3.6.0`
+- [eslint] Updated `eslint-import-resolver-webpack` from `0.13.2` to `0.13.4`
+- [eslint] Updated `eslint-plugin-functional` from `5.0.8` to `6.0.0`
+- [eslint] Updated `eslint-plugin-jsdoc` from `46.4.5` to `46.4.6`
+- [eslint] Updated `eslint-plugin-react` from `7.33.0` to `7.33.1`
+- [stylelint] Updated `stylelint-scss` from `5.0.1` to `5.1.0`
+- [eslint] Removed deprecated (and currently disabled) `no-return-await` rule
+- [styleint] Added but disabled [`scss/function-disallowed-list`](https://github.com/stylelint-scss/stylelint-scss/blob/master/src/rules/function-disallowed-list) rule
+
+[Show all code changes](https://github.com/jens-duttke/linter-bundle/compare/v3.5.0...v3.6.0)
+
+## [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',
@@ -294,7 +294,6 @@ module.exports = {
 					...ensureType.array(global.linterBundleSettings?.overrides?.general?.['no-restricted-syntax']?.additionalRestrictions)
 				],
 				'no-return-assign': 'error',
-				'no-return-await': 'off', // Covered by @typescript-eslint/return-await
 				'no-script-url': 'error',
 				'no-self-assign': 'error',
 				'no-self-compare': 'error',
@@ -423,9 +422,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 +743,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
@@ -784,10 +782,10 @@ module.exports = {
 				 * @see https://github.com/jonaskello/eslint-plugin-functional#supported-rules
 				 */
 				'functional/functional-parameters': 'off',
-				'functional/immutable-data': 'off', // This rule would require a lot of additional code and workarounds, which would make the result much more illegible.
+				'functional/immutable-data': 'off', // This rule would require a lot of additional code and workarounds, which would make the result much more illegible // @todo is that resolved in v6.0.0?
 				'functional/no-classes': 'off',
 				'functional/no-conditional-statements': 'off',
-				'functional/no-expression-statements': ['off', { ignoreVoid: true }], // Creates too much false-positives
+				'functional/no-expression-statements': ['off', { ignoreVoid: true }], // Creates too much false-positives // @todo is that resolved in v6.0.0?
 				'functional/no-let': 'off', // This is better covered by the `prefer-const` rule
 				'functional/no-loop-statements': 'off',
 				'functional/no-mixed-types': ['error', {
@@ -961,7 +959,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.6.0",
 	"description": "Ready-to use bundle of linting tools, containing configurations for ESLint, stylelint and markdownlint.",
 	"keywords": [
 		"eslint",
@@ -40,40 +40,40 @@
 		"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",
-		"eslint": "8.45.0",
-		"eslint-import-resolver-typescript": "3.5.5",
-		"eslint-import-resolver-webpack": "0.13.2",
+		"@typescript-eslint/eslint-plugin": "6.3.0",
+		"@typescript-eslint/parser": "6.3.0",
+		"@typescript-eslint/utils": "6.3.0",
+		"eslint": "8.47.0",
+		"eslint-import-resolver-typescript": "3.6.0",
+		"eslint-import-resolver-webpack": "0.13.4",
 		"eslint-plugin-eslint-comments": "3.2.0",
-		"eslint-plugin-functional": "5.0.8",
-		"eslint-plugin-import": "2.27.5",
+		"eslint-plugin-functional": "6.0.0",
+		"eslint-plugin-import": "2.28.0",
 		"eslint-plugin-jest": "27.2.3",
-		"eslint-plugin-jsdoc": "46.4.4",
+		"eslint-plugin-jsdoc": "46.4.6",
 		"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.1",
 		"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",
 		"stylelint": "15.10.2",
 		"stylelint-declaration-block-no-ignored-properties": "2.7.0",
 		"stylelint-order": "6.0.3",
-		"stylelint-scss": "5.0.1",
+		"stylelint-scss": "5.1.0",
 		"stylelint-use-logical-spec": "5.0.0"
 	},
 	"peerDependencies": {
 		"postcss-value-parser": "*",
-		"typescript": "^4.0.0"
+		"typescript": ">=4.0.0"
 	},
 	"devDependencies": {
-		"@types/eslint": "8.44.0",
+		"@types/eslint": "8.44.2",
 		"@types/micromatch": "4.0.2",
-		"@types/node": "20.4.2",
+		"@types/node": "20.4.9",
 		"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,8 @@ 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-disallowed-list': null,
+		'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',