linter-bundle 3.5.0 → 3.7.0

package/CHANGELOG.md

@@ -6,7 +6,29 @@ 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.5.0...HEAD)
+[Show all code changes](https://github.com/jens-duttke/linter-bundle/compare/v3.7.0...HEAD)
+
+## [3.7.0] - 2023-08-11
+
+- [eslint] Adjusted `jsdoc/tag-lines` configuration to enforce no empty like for `@typedef`, `@property` and `@returns`; and weakened line configuration for `@see`
+
+[Show all code changes](https://github.com/jens-duttke/linter-bundle/compare/v3.6.0...v3.7.0)
+
+## [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
 

package/eslint/index.js

@@ -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',
@@ -783,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', {

package/eslint/overrides-jsdoc.js

@@ -70,13 +70,16 @@ module.exports = {
 						file: { lines: 'always' },
 						fileOverview: { lines: 'always' },
 						license: { lines: 'always' },
-						see: { lines: 'never' },
-						template: { lines: 'never' },
 						param: { lines: 'never' },
 						private: { lines: 'never' },
+						property: { lines: 'never' },
 						protected: { lines: 'never' },
 						public: { lines: 'never' },
-						throws: { lines: 'never' }
+						returns: { lines: 'never' },
+						see: { lines: 'any' },
+						template: { lines: 'never' },
+						throws: { lines: 'never' },
+						typedef: { lines: 'never' }
 					},
 					startLines: null,
 					endLines: null,

package/lint.js

@@ -259,7 +259,6 @@ void (async () => {
  *
  * @param {ReturnType<isNpmOrYarn>} npmOrYarn - This should be the return value of `isNpmOrYarn()`.
  * @returns {boolean} Returns `true` if the environment is valid, otherwise `false` is returned.
- *
  */
 function validateEnvironment (npmOrYarn) {
 	const outdatedOverrides = validatePackageOverrides();
@@ -312,7 +311,6 @@ function validateEnvironment (npmOrYarn) {
  *
  * @param {string[]} argv - Command-line arguments (usual `process.argv.splice(2)`)
  * @returns {TaskNameAndConfig[]} The task execution setup.
- *
  * @throws {Error} If no task has be specified in the arguments.
  */
 function getTasksToRun (argv) {
@@ -379,7 +377,6 @@ function getTasksToRun (argv) {
  * @param {string} pattern - Glob pattern
  * @param {Partial<Record<string, (string | true)[]>>} config - Linter configuration
  * @returns {string} Space-separated file names in double-quotes to be used in the command-line, or an empty string if no file matches.
- *
  */
 function getIncludes (list, pattern, config) {
 	const include = config['include']?.[0];
@@ -402,7 +399,6 @@ function getIncludes (list, pattern, config) {
  *
  * @param {TaskSetup} setup - The task execution setup.
  * @returns {Job} Job
- *
  */
 function runTask (setup) {
 	return {
@@ -419,7 +415,6 @@ function runTask (setup) {
  * @param {Partial<Record<string, (string | true)[]>>} config - The configuration of the task.
  * @param {{ code?: number; stdout?: string; stderr?: string; }} output - The output which should be returned as result of the job.
  * @returns {Job} Job
- *
  */
 function generateDummyJobOutput (taskName, config, output) {
 	return {
@@ -448,7 +443,6 @@ function generateDummyJobOutput (taskName, config, output) {
  *
  * @param {TaskSetup} setup - The task execution setup.
  * @returns {string} The title of the job with a leading line-break and two trailing line-breaks.
- *
  */
 function getJobTitle (setup) {
 	/** @type {string} */

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "linter-bundle",
-	"version": "3.5.0",
+	"version": "3.7.0",
 	"description": "Ready-to use bundle of linting tools, containing configurations for ESLint, stylelint and markdownlint.",
 	"keywords": [
 		"eslint",
@@ -40,21 +40,21 @@
 		"check-outdated": "npx --yes -- check-outdated --ignore-pre-releases"
 	},
 	"dependencies": {
-		"@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",
+		"@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-functional": "6.0.0",
 		"eslint-plugin-import": "2.28.0",
 		"eslint-plugin-jest": "27.2.3",
-		"eslint-plugin-jsdoc": "46.4.5",
+		"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.33.0",
+		"eslint-plugin-react": "7.33.1",
 		"eslint-plugin-react-hooks": "4.6.0",
 		"eslint-plugin-unicorn": "48.0.1",
 		"markdownlint-cli": "0.35.0",
@@ -63,17 +63,17 @@
 		"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.1",
+		"@types/eslint": "8.44.2",
 		"@types/micromatch": "4.0.2",
-		"@types/node": "20.4.5",
+		"@types/node": "20.4.9",
 		"stylelint-find-new-rules": "4.1.0",
 		"typescript": "5.1.6"
 	}

package/stylelint/index.js

@@ -921,6 +921,7 @@ module.exports = {
 		'scss/double-slash-comment-inline': null,
 		'scss/double-slash-comment-whitespace-inside': 'always',
 		'scss/function-color-relative': 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,

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

@@ -139,11 +139,11 @@ const getDisallowedList = (ignore) => {
 /**
  * Returns the input string stripped of its vendor prefix.
  *
- * @param {string} property - String with or without vendor prefix.
- * @returns {string} String name without vendor prefixes.
- *
  * @example
  * unprefixed('-moz-tab-size') //=> 'tab-size'
+ *
+ * @param {string} property - String with or without vendor prefix.
+ * @returns {string} String name without vendor prefixes.
  */
 const unprefixed = (property) => property.replace(/^-\w+-/u, '');