From 228c72531b30c0ff7759630222d815386837a85c Mon Sep 17 00:00:00 2001 From: Kristjan ESPERANTO <35647502+KristjanESPERANTO@users.noreply.github.com> Date: Tue, 6 Feb 2024 22:57:16 +0100 Subject: [PATCH] linting: add rules based on @stylistic/eslint-plugin Co-Authored-By: Jannis R --- .eslintrc.json | 30 ++++++++++++++++++++++++------ package.json | 4 +++- 2 files changed, 27 insertions(+), 7 deletions(-) diff --git a/.eslintrc.json b/.eslintrc.json index 4fd76634..45c9cf2f 100644 --- a/.eslintrc.json +++ b/.eslintrc.json @@ -3,17 +3,15 @@ "es2021": true, "node": true }, - "extends": "eslint:recommended", - "globals": { - "Atomics": "readonly", - "SharedArrayBuffer": "readonly" - }, + "extends": ["eslint:recommended", "plugin:@stylistic/all-extends"], "ignorePatterns": ["node_modules", "*example.js"], "parserOptions": { "ecmaVersion": 2021, "sourceType": "module" }, "rules": { + "curly": "error", + "no-implicit-coercion": "error", "no-unused-vars": [ "error", { @@ -21,7 +19,27 @@ "args": "none", "ignoreRestSiblings": false } - ] + ], + "@stylistic/array-bracket-newline": ["error", "consistent"], + "@stylistic/array-element-newline": ["error", "consistent"], + "@stylistic/arrow-parens": "off", + "@stylistic/comma-dangle": ["error", "always-multiline"], + "@stylistic/dot-location": ["error", "property"], + "@stylistic/function-call-argument-newline": ["error", "consistent"], + "@stylistic/function-paren-newline": "off", + "@stylistic/indent": ["error", "tab"], + "@stylistic/indent-binary-ops": ["error", "tab"], + "@stylistic/max-len": "off", + "@stylistic/multiline-ternary": ["error", "always-multiline"], + "@stylistic/newline-per-chained-call": ["error", { "ignoreChainWithDepth": 1 }], + "@stylistic/no-mixed-operators": "off", + "@stylistic/no-tabs": "off", + "@stylistic/object-property-newline": "off", + "@stylistic/one-var-declaration-per-line": "off", + "@stylistic/operator-linebreak": ["error", "before"], + "@stylistic/padded-blocks": "off", + "@stylistic/quote-props": ["error", "consistent-as-needed"], + "@stylistic/quotes": ["error", "single"] }, "overrides": [ { diff --git a/package.json b/package.json index 3798e480..d5f3c5b3 100644 --- a/package.json +++ b/package.json @@ -66,7 +66,8 @@ "@pollyjs/adapter-node-http": "^6.0.5", "@pollyjs/core": "^6.0.5", "@pollyjs/persister-fs": "^6.0.5", - "eslint": "^8.15.0", + "@stylistic/eslint-plugin": "^1.5.1", + "eslint": "^8.56.0", "is-coordinates": "^2.0.2", "is-roughly-equal": "^0.1.0", "tap": "^18.0.4", @@ -74,6 +75,7 @@ }, "scripts": { "lint": "eslint .", + "lint:fix": "eslint . --fix", "test-unit": "tap test/lib/*.js test/*.js test/format/*.js test/parse/*.js", "test-integration": "VCR_MODE=playback tap test/e2e/*.js", "test-integration:record": "VCR_MODE=record tap -t60 -j1 test/e2e/*.js",