linter-bundle 2.3.1 → 2.4.0

package/CHANGELOG.md

@@ -6,7 +6,25 @@ 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/v2.3.1...HEAD)
+[Show all code changes](https://github.com/jens-duttke/linter-bundle/compare/v2.4.0...HEAD)
+
+## [2.4.0] - 2022.02.09
+
+- [eslint] Updated `@typescript-eslint/eslint-plugin` from v5.10.1 to v5.11.0
+- [eslint] Updated `eslint` from v8.7.0 to v8.8.0
+- [eslint] Updated `eslint-plugin-functional` from v4.1.1 to v4.2.0
+- [eslint] Updated `eslint-plugin-jest` from v26.0.0 to v26.1.0
+- [eslint] Updated `eslint-plugin-jsdoc` from v37.7.0 to v37.8.1
+- [markdownlint] Updated `markdownlint-cli` from v0.30.0 to v0.31.1, which adds the new rules `MD049` and `MD050` for consistent emphasis/strong style, and improves the rules `MD007`/`MD010`/`MD032`/`MD033`/`MD035`/`MD037`/`MD039`
+- [stylelint] Updated `stylelint` from v14.3.0 to v14.4.0
+- [eslint] Activated `unusedExports` option of the [`import/no-unused-modules`](https://github.com/import-js/eslint-plugin-import/blob/main/docs/rules/no-unused-modules.md) rule
+- [eslint] Activate [`@typescript-eslint/explicit-function-return-type`](https://github.com/typescript-eslint/typescript-eslint/blob/master/packages/eslint-plugin/docs/rules/explicit-function-return-type.md) rule
+- [eslint/react] Add React class methods (`componentDidMount`, `render` etc.) to the allowed names in the [`@typescript-eslint/explicit-function-return-type`](https://github.com/typescript-eslint/typescript-eslint/blob/master/packages/eslint-plugin/docs/rules/explicit-function-return-type.md) rule
+- [eslint/jest] Make use of new [`no-conditional-in-test`](https://github.com/jest-community/eslint-plugin-jest/blob/main/docs/rules/no-conditional-in-test.md) rule, which deprecates `no-if`
+- [eslint/jest] Make use of new [`prefer-snapshot-hint`](https://github.com/jest-community/eslint-plugin-jest/blob/main/docs/rules/prefer-snapshot-hint.md) rule
+- [stylelint] Added but disabled new [`function-no-unknown`](https://github.com/stylelint/stylelint/blob/main/lib/rules/function-no-unknown/README.md) rule, because of false positives
+
+[Show all code changes](https://github.com/jens-duttke/linter-bundle/compare/v2.3.1...v2.4.0)
 
 ## [2.3.1] - 2022.01.28
 

package/eslint/index.js

@@ -424,7 +424,7 @@ module.exports = {
 		'@typescript-eslint/consistent-type-exports': 'error',
 		'@typescript-eslint/consistent-type-imports': ['error', { disallowTypeAnnotations: false }],
 		'@typescript-eslint/dot-notation': 'error',
-		'@typescript-eslint/explicit-function-return-type': 'off', // @todo Disabled till we can specify exceptions (React-default class methods, like render())
+		'@typescript-eslint/explicit-function-return-type': 'error',
 		'@typescript-eslint/explicit-member-accessibility': ['error', {
 			ignoredMethodNames: [
 				'constructor',
@@ -789,7 +789,7 @@ module.exports = {
 			]
 		}],
 		'import/no-unresolved': ['error', { caseSensitiveStrict: true }],
-		'import/no-unused-modules': 'error',
+		'import/no-unused-modules': ['error', { unusedExports: true }],
 		'import/no-useless-path-segments': 'error',
 		'import/no-webpack-loader-syntax': 'off', // Indeed, you should avoid that, but if we do it, we have a reason for it
 		'import/order': ['error', {

package/eslint/overrides-jest.js

@@ -66,6 +66,7 @@ module.exports = {
 				'jest/no-alias-methods': 'error',
 				'jest/no-commented-out-tests': 'error',
 				'jest/no-conditional-expect': 'error',
+				'jest/no-conditional-in-test': 'error',
 				'jest/no-deprecated-functions': 'error',
 				'jest/no-disabled-tests': 'error',
 				'jest/no-done-callback': 'error',
@@ -74,7 +75,6 @@ module.exports = {
 				'jest/no-focused-tests': 'error',
 				'jest/no-hooks': ['error', { allow: ['beforeEach', 'afterEach', 'afterAll'] }],
 				'jest/no-identical-title': 'error',
-				'jest/no-if': 'error',
 				'jest/no-interpolation-in-snapshots': 'error',
 				'jest/no-jasmine-globals': 'error',
 				'jest/no-jest-import': 'error',
@@ -90,6 +90,7 @@ module.exports = {
 				'jest/require-hook': 'error',
 				'jest/prefer-comparison-matcher': 'error',
 				'jest/prefer-equality-matcher': 'error',
+				'jest/prefer-snapshot-hint': 'error',
 				'jest/prefer-spy-on': 'error',
 				'jest/prefer-strict-equal': 'error',
 				'jest/prefer-to-be': 'error',

package/eslint/overrides-react.js

@@ -15,6 +15,19 @@ module.exports = {
 				 * @see https://github.com/typescript-eslint/typescript-eslint/blob/master/packages/eslint-plugin/docs/rules
 				 */
 				'@typescript-eslint/member-ordering': 'off', // For React components we are using react/sort-comp
+				'@typescript-eslint/explicit-function-return-type': ['error', {
+					allowedNames: [
+						'componentDidCatch',
+						'componentDidMount',
+						'componentDidUpdate',
+						'componentWillUnmount',
+						'getDerivedStateFromError',
+						'getDerivedStateFromProps',
+						'getSnapshotBeforeUpdate',
+						'render',
+						'shouldComponentUpdate'
+					]
+				}],
 
 				/**
 				 * eslint-plugin-react-hooks

package/eslint/overrides-type-declarations.js

@@ -21,6 +21,7 @@ module.exports = {
 				 * @see https://github.com/import-js/eslint-plugin-import
 				 */
 				'import/no-default-export': 'off',
+				'import/no-unused-modules': 'off',
 
 				/**
 				 * eslint-plugin-unicorn

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "linter-bundle",
-	"version": "2.3.1",
+	"version": "2.4.0",
 	"description": "Ready-to use bundle of linting tools, containing configurations for ESLint, stylelint and markdownlint.",
 	"keywords": [
 		"eslint",
@@ -39,25 +39,25 @@
 		"check-outdated": "npx --yes -- check-outdated --ignore-pre-releases"
 	},
 	"dependencies": {
-		"@typescript-eslint/eslint-plugin": "5.10.1",
-		"@typescript-eslint/parser": "5.10.1",
-		"eslint": "8.7.0",
+		"@typescript-eslint/eslint-plugin": "5.11.0",
+		"@typescript-eslint/parser": "5.11.0",
+		"eslint": "8.8.0",
 		"eslint-import-resolver-typescript": "2.5.0",
 		"eslint-import-resolver-webpack": "0.13.2",
 		"eslint-plugin-eslint-comments": "3.2.0",
-		"eslint-plugin-functional": "4.1.1",
+		"eslint-plugin-functional": "4.2.0",
 		"eslint-plugin-import": "2.25.4",
-		"eslint-plugin-jest": "26.0.0",
-		"eslint-plugin-jsdoc": "37.7.0",
+		"eslint-plugin-jest": "26.1.0",
+		"eslint-plugin-jsdoc": "37.8.1",
 		"eslint-plugin-jsx-a11y": "6.5.1",
 		"eslint-plugin-node": "11.1.0",
 		"eslint-plugin-promise": "6.0.0",
 		"eslint-plugin-react": "7.28.0",
 		"eslint-plugin-react-hooks": "4.3.0",
 		"eslint-plugin-unicorn": "40.1.0",
-		"markdownlint-cli": "0.30.0",
+		"markdownlint-cli": "0.31.1",
 		"postcss-scss": "4.0.3",
-		"stylelint": "14.3.0",
+		"stylelint": "14.4.0",
 		"stylelint-declaration-block-no-ignored-properties": "2.5.0",
 		"stylelint-order": "5.0.0",
 		"stylelint-scss": "4.1.0",
@@ -70,7 +70,7 @@
 	},
 	"devDependencies": {
 		"@types/eslint": "8.4.1",
-		"@types/node": "17.0.13",
+		"@types/node": "17.0.16",
 		"stylelint-find-new-rules": "^3.0.4",
 		"typescript": "4.5.5"
 	}

package/stylelint/index.js

@@ -183,6 +183,7 @@ module.exports = {
 		'function-linear-gradient-no-nonstandard-direction': true,
 		'function-max-empty-lines': 0,
 		'function-name-case': 'lower',
+		'function-no-unknown': false, // @todo Create issue for false positives for "color.scale"
 		'function-parentheses-newline-inside': 'always-multi-line',
 		'function-parentheses-space-inside': 'never-single-line',
 		'function-url-no-scheme-relative': true,