@aarongoldenthal/eslint-config-standard 44.0.1 → 45.0.1
This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
- package/README.md +0 -1
- package/base-config.js +3 -1
- package/node-config.js +2 -0
- package/package.json +19 -21
- package/playwright-config.js +4 -0
- package/recommended-esm-base.js +0 -2
- package/recommended.js +0 -2
- package/unicorn-config.js +1 -0
- package/vitest-config.js +1 -0
- package/promise-config.js +0 -32
package/README.md
CHANGED
|
@@ -13,7 +13,6 @@ configurations compatible with ESLint v9+ for the following:
|
|
|
13
13
|
| `eslint-plugin-jsdoc` | [`jsdoc-config`](./jsdoc-config.js) | `jsdoc` |
|
|
14
14
|
| `eslint-plugin-n` | [`node-config`](./node-config.js) | `node` |
|
|
15
15
|
| `eslint-plugin-playwright` | [`playwright-config`](./playwright-config.js) | `playwright` |
|
|
16
|
-
| `eslint-plugin-promise` | [`promise-config`](./promise-config.js) | `promise` |
|
|
17
16
|
| `eslint-plugin-unicorn` | [`unicorn-config`](./unicorn-config.js), [`esm-config`](./esm-config.js) | `unicorn` |
|
|
18
17
|
| `eslint-plugin-vitest` | [`vitest-config`](./vitest-config.js) | `vitest` |
|
|
19
18
|
| `typescript-eslint` | [`typescript-eslint-config`](./typescript-eslint-config.js) | `@typescript-eslint` |
|
package/base-config.js
CHANGED
|
@@ -227,6 +227,7 @@ const configs = [
|
|
|
227
227
|
'no-shadow-restricted-names': 'error', // Recommended
|
|
228
228
|
'no-ternary': 'off',
|
|
229
229
|
'no-throw-literal': 'error',
|
|
230
|
+
'no-unassigned-vars': 'error', // Recommended
|
|
230
231
|
'no-undef-init': 'error',
|
|
231
232
|
// Disabled, too many valid positives
|
|
232
233
|
'no-undefined': 'off',
|
|
@@ -237,6 +238,7 @@ const configs = [
|
|
|
237
238
|
{ allowShortCircuit: true, allowTernary: true }
|
|
238
239
|
],
|
|
239
240
|
'no-unused-labels': 'error', // Recommended
|
|
241
|
+
'no-useless-assignment': 'error', // Recommended
|
|
240
242
|
'no-useless-call': 'error',
|
|
241
243
|
'no-useless-catch': 'error', // Recommended
|
|
242
244
|
'no-useless-computed-key': 'error',
|
|
@@ -269,7 +271,7 @@ const configs = [
|
|
|
269
271
|
'prefer-spread': 'error',
|
|
270
272
|
'prefer-template': 'error',
|
|
271
273
|
'preserve-caught-error': 'error',
|
|
272
|
-
radix:
|
|
274
|
+
radix: 'error',
|
|
273
275
|
'require-await': 'error',
|
|
274
276
|
'require-unicode-regexp': 'off',
|
|
275
277
|
'require-yield': 'error', // Recommended
|
package/node-config.js
CHANGED
|
@@ -52,9 +52,11 @@ const config = defineConfig({
|
|
|
52
52
|
'node/no-unsupported-features/node-builtins': 'error',
|
|
53
53
|
'node/prefer-global/buffer': ['error', 'always'],
|
|
54
54
|
'node/prefer-global/console': ['error', 'always'],
|
|
55
|
+
'node/prefer-global/crypto': ['error', 'always'],
|
|
55
56
|
'node/prefer-global/process': ['error', 'always'],
|
|
56
57
|
'node/prefer-global/text-decoder': ['error', 'always'],
|
|
57
58
|
'node/prefer-global/text-encoder': ['error', 'always'],
|
|
59
|
+
'node/prefer-global/timers': ['error', 'always'],
|
|
58
60
|
'node/prefer-global/url': ['error', 'always'],
|
|
59
61
|
'node/prefer-global/url-search-params': ['error', 'always'],
|
|
60
62
|
'node/prefer-promises/dns': 'error',
|
package/package.json
CHANGED
|
@@ -1,6 +1,6 @@
|
|
|
1
1
|
{
|
|
2
2
|
"name": "@aarongoldenthal/eslint-config-standard",
|
|
3
|
-
"version": "
|
|
3
|
+
"version": "45.0.1",
|
|
4
4
|
"description": "Standard ESLint configuration settings",
|
|
5
5
|
"type": "module",
|
|
6
6
|
"scripts": {
|
|
@@ -35,7 +35,6 @@
|
|
|
35
35
|
"./jsdoc-config": "./jsdoc-config.js",
|
|
36
36
|
"./node-config": "./node-config.js",
|
|
37
37
|
"./playwright-config": "./playwright-config.js",
|
|
38
|
-
"./promise-config": "./promise-config.js",
|
|
39
38
|
"./recommended": "./recommended.js",
|
|
40
39
|
"./recommended-esm": "./recommended-esm.js",
|
|
41
40
|
"./recommended-ts": "./recommended-ts.js",
|
|
@@ -53,31 +52,30 @@
|
|
|
53
52
|
},
|
|
54
53
|
"homepage": "https://gitlab.com/gitlab-ci-utils/eslint-config-standard",
|
|
55
54
|
"dependencies": {
|
|
56
|
-
"@eslint-community/eslint-plugin-eslint-comments": "4.
|
|
57
|
-
"@vitest/eslint-plugin": "1.6.
|
|
55
|
+
"@eslint-community/eslint-plugin-eslint-comments": "4.7.1",
|
|
56
|
+
"@vitest/eslint-plugin": "1.6.14",
|
|
58
57
|
"eslint-config-prettier": "10.1.8",
|
|
59
|
-
"eslint-plugin-jest": "29.
|
|
60
|
-
"eslint-plugin-jsdoc": "62.
|
|
61
|
-
"eslint-plugin-n": "17.
|
|
62
|
-
"eslint-plugin-playwright": "2.
|
|
63
|
-
"eslint-plugin-
|
|
64
|
-
"
|
|
65
|
-
"
|
|
66
|
-
"typescript-eslint": "8.53.1"
|
|
58
|
+
"eslint-plugin-jest": "29.15.1",
|
|
59
|
+
"eslint-plugin-jsdoc": "62.8.1",
|
|
60
|
+
"eslint-plugin-n": "17.24.0",
|
|
61
|
+
"eslint-plugin-playwright": "2.10.1",
|
|
62
|
+
"eslint-plugin-unicorn": "63.0.0",
|
|
63
|
+
"globals": "17.4.0",
|
|
64
|
+
"typescript-eslint": "8.58.0"
|
|
67
65
|
},
|
|
68
66
|
"devDependencies": {
|
|
69
|
-
"@eslint/config-inspector": "1.
|
|
70
|
-
"@vitest/coverage-v8": "4.
|
|
71
|
-
"eslint": "
|
|
72
|
-
"jest": "30.
|
|
73
|
-
"markdownlint-cli2": "0.
|
|
67
|
+
"@eslint/config-inspector": "1.5.0",
|
|
68
|
+
"@vitest/coverage-v8": "4.1.2",
|
|
69
|
+
"eslint": "10.1.0",
|
|
70
|
+
"jest": "30.3.0",
|
|
71
|
+
"markdownlint-cli2": "0.22.0",
|
|
74
72
|
"prettier": "3.8.1",
|
|
75
|
-
"typescript": "
|
|
76
|
-
"vitest": "4.
|
|
73
|
+
"typescript": "6.0.2",
|
|
74
|
+
"vitest": "4.1.2"
|
|
77
75
|
},
|
|
78
76
|
"peerDependencies": {
|
|
79
|
-
"eslint": "^
|
|
80
|
-
"typescript": "^5.8.0"
|
|
77
|
+
"eslint": "^10.0.0",
|
|
78
|
+
"typescript": "^5.8.0 || ^6.0.0"
|
|
81
79
|
},
|
|
82
80
|
"peerDependenciesMeta": {
|
|
83
81
|
"typescript": {
|
package/playwright-config.js
CHANGED
|
@@ -24,6 +24,7 @@ export default defineConfig({
|
|
|
24
24
|
'playwright/no-conditional-expect': 'error',
|
|
25
25
|
'playwright/no-conditional-in-test': 'error',
|
|
26
26
|
'playwright/no-duplicate-hooks': 'error',
|
|
27
|
+
'playwright/no-duplicate-slow': 'error',
|
|
27
28
|
'playwright/no-element-handle': 'error',
|
|
28
29
|
'playwright/no-eval': 'error',
|
|
29
30
|
'playwright/no-focused-test': 'error',
|
|
@@ -37,6 +38,7 @@ export default defineConfig({
|
|
|
37
38
|
'playwright/no-raw-locators': 'error',
|
|
38
39
|
'playwright/no-restricted-locators': 'off',
|
|
39
40
|
'playwright/no-restricted-matchers': 'off',
|
|
41
|
+
'playwright/no-restricted-roles': 'off',
|
|
40
42
|
'playwright/no-skipped-test': 'error',
|
|
41
43
|
'playwright/no-slowed-test': 'error',
|
|
42
44
|
'playwright/no-standalone-expect': 'error',
|
|
@@ -62,6 +64,8 @@ export default defineConfig({
|
|
|
62
64
|
'playwright/prefer-web-first-assertions': 'error',
|
|
63
65
|
'playwright/require-hook': 'error',
|
|
64
66
|
'playwright/require-soft-assertions': 'off',
|
|
67
|
+
'playwright/require-tags': 'off',
|
|
68
|
+
'playwright/require-to-pass-timeout': 'error',
|
|
65
69
|
'playwright/require-to-throw-message': 'error',
|
|
66
70
|
'playwright/require-top-level-describe': 'error',
|
|
67
71
|
'playwright/valid-describe-callback': 'error',
|
package/recommended-esm-base.js
CHANGED
|
@@ -4,7 +4,6 @@ import esmConfig from './esm-config.js';
|
|
|
4
4
|
import jsdocConfig from './jsdoc-config.js';
|
|
5
5
|
import nodeConfig from './node-config.js';
|
|
6
6
|
import playwrightConfig from './playwright-config.js';
|
|
7
|
-
import promiseConfig from './promise-config.js';
|
|
8
7
|
import unicornConfig from './unicorn-config.js';
|
|
9
8
|
import vitestConfig from './vitest-config.js';
|
|
10
9
|
|
|
@@ -14,7 +13,6 @@ const esmBaseConfig = [
|
|
|
14
13
|
jsdocConfig,
|
|
15
14
|
nodeConfig,
|
|
16
15
|
playwrightConfig,
|
|
17
|
-
promiseConfig,
|
|
18
16
|
unicornConfig,
|
|
19
17
|
vitestConfig,
|
|
20
18
|
baseConfig,
|
package/recommended.js
CHANGED
|
@@ -7,7 +7,6 @@ import jsdocConfig from './jsdoc-config.js';
|
|
|
7
7
|
import nodeConfig from './node-config.js';
|
|
8
8
|
import playwrightConfig from './playwright-config.js';
|
|
9
9
|
import prettierConfig from 'eslint-config-prettier';
|
|
10
|
-
import promiseConfig from './promise-config.js';
|
|
11
10
|
import unicornConfig from './unicorn-config.js';
|
|
12
11
|
|
|
13
12
|
// Create local variable due to vitest v4 coverage changes
|
|
@@ -17,7 +16,6 @@ const config = defineConfig([
|
|
|
17
16
|
jsdocConfig,
|
|
18
17
|
nodeConfig,
|
|
19
18
|
playwrightConfig,
|
|
20
|
-
promiseConfig,
|
|
21
19
|
unicornConfig,
|
|
22
20
|
baseConfig,
|
|
23
21
|
{ name: 'prettier (all files)', ...prettierConfig }
|
package/unicorn-config.js
CHANGED
|
@@ -25,6 +25,7 @@ const config = defineConfig([
|
|
|
25
25
|
'unicorn/explicit-length-check': 'error', // Recommended
|
|
26
26
|
'unicorn/filename-case': ['error', { case: 'kebabCase' }], // Recommended
|
|
27
27
|
'unicorn/import-style': 'error', // Recommended
|
|
28
|
+
'unicorn/isolated-functions': 'error', // Recommended
|
|
28
29
|
'unicorn/new-for-builtins': 'error', // Recommended
|
|
29
30
|
'unicorn/no-abusive-eslint-disable': 'error', // Recommended
|
|
30
31
|
'unicorn/no-accessor-recursion': 'error', // Recommended
|
package/vitest-config.js
CHANGED
|
@@ -97,6 +97,7 @@ const config = defineConfig([
|
|
|
97
97
|
'vitest/require-test-timeout': 'off',
|
|
98
98
|
'vitest/require-to-throw-message': 'error',
|
|
99
99
|
'vitest/require-top-level-describe': 'error',
|
|
100
|
+
'vitest/unbound-method': 'off',
|
|
100
101
|
'vitest/valid-describe-callback': 'error', // Recommended
|
|
101
102
|
'vitest/valid-expect': 'error', // Recommended
|
|
102
103
|
'vitest/valid-expect-in-promise': 'error', // Recommended
|
package/promise-config.js
DELETED
|
@@ -1,32 +0,0 @@
|
|
|
1
|
-
import { defineConfig } from 'eslint/config';
|
|
2
|
-
import { filePatterns } from './settings.js';
|
|
3
|
-
import promisePlugin from 'eslint-plugin-promise';
|
|
4
|
-
|
|
5
|
-
// Create local variable due to vitest v4 coverage changes
|
|
6
|
-
const config = defineConfig({
|
|
7
|
-
files: [...filePatterns.allJs, ...filePatterns.allTs],
|
|
8
|
-
name: 'promise (all files)',
|
|
9
|
-
plugins: { promise: promisePlugin },
|
|
10
|
-
rules: {
|
|
11
|
-
'promise/always-return': 'error',
|
|
12
|
-
'promise/avoid-new': 'error',
|
|
13
|
-
'promise/catch-or-return': 'error',
|
|
14
|
-
'promise/no-callback-in-promise': 'error',
|
|
15
|
-
'promise/no-multiple-resolved': 'error',
|
|
16
|
-
'promise/no-native': 'off',
|
|
17
|
-
'promise/no-nesting': 'error',
|
|
18
|
-
'promise/no-new-statics': 'error',
|
|
19
|
-
'promise/no-promise-in-callback': 'error',
|
|
20
|
-
'promise/no-return-in-finally': 'error',
|
|
21
|
-
'promise/no-return-wrap': 'error',
|
|
22
|
-
'promise/param-names': [
|
|
23
|
-
'error',
|
|
24
|
-
{ rejectPattern: '^reject$', resolvePattern: '^resolve$' }
|
|
25
|
-
],
|
|
26
|
-
'promise/prefer-await-to-callbacks': 'error',
|
|
27
|
-
'promise/prefer-await-to-then': 'error',
|
|
28
|
-
'promise/valid-params': 'error'
|
|
29
|
-
}
|
|
30
|
-
});
|
|
31
|
-
|
|
32
|
-
export default config;
|