-
Notifications
You must be signed in to change notification settings - Fork 5
/
.eslintrc.js
110 lines (108 loc) · 3.65 KB
/
.eslintrc.js
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
// https://github.com/typescript-eslint/typescript-eslint/blob/master/docs/getting-started/linting/README.md
module.exports = {
root: true,
parser: '@typescript-eslint/parser',
plugins: ['@typescript-eslint', 'eslint-plugin-tsdoc'],
extends: ['eslint:recommended', 'plugin:@typescript-eslint/recommended', 'prettier'],
rules: {
'tsdoc/syntax': 'warn',
// Problems
'array-callback-return': [
'error',
{
allowImplicit: false,
checkForEach: true,
},
],
'no-constant-binary-expression': 'error',
'no-constructor-return': 'error',
'no-duplicate-imports': 'error',
'no-promise-executor-return': 'error',
'no-self-compare': 'error',
'no-template-curly-in-string': 'error',
'no-unmodified-loop-condition': 'error',
'no-unreachable-loop': 'error',
'no-unused-private-class-members': 'error',
'require-atomic-updates': 'error',
'@typescript-eslint/no-unused-vars': [
'warn',
{
argsIgnorePattern: '^_',
},
],
// Suggestions
'capitalized-comments': ['error', 'always', { ignoreConsecutiveComments: true }],
complexity: ['error', 25], // TODO: Should be lower (15?), but validator has complex methods
curly: 'error',
'default-case-last': 'error',
'default-param-last': 'error',
eqeqeq: ['error', 'smart'],
'func-names': 'error',
'func-style': ['error', 'declaration', { allowArrowFunctions: true }],
'grouped-accessor-pairs': ['error', 'getBeforeSet'],
'max-depth': ['error', 4], // TODO: Should be lower (3), but validator has complex methods
'max-nested-callbacks': ['error', 3],
'max-params': ['error', 4], // TODO: Should be lower (3), but validator has complex methods,
'new-cap': ['error', { newIsCapExceptions: ['ctor'] }],
'no-caller': 'error',
'no-confusing-arrow': [
'error',
{
allowParens: true,
},
],
'no-console': ['warn'], // We want this to debug, but the check workflow should accept no warns
'no-div-regex': 'error',
'no-eval': 'error',
'no-extend-native': 'error',
'no-extra-bind': 'error',
'no-extra-label': 'error',
'no-extra-semi': 'error',
'no-floating-decimal': 'error',
'no-implied-eval': 'error',
'no-invalid-this': 'error',
'no-labels': 'error',
'no-lonely-if': 'error',
'no-multi-assign': 'error',
'no-multi-str': 'error',
'no-nested-ternary': 'error',
'no-new': 'error',
'no-new-func': 'error',
'no-new-object': 'error',
'no-new-wrappers': 'error',
'no-param-reassign': 'error',
'no-proto': 'error',
'no-return-assign': 'error',
'no-return-await': 'error',
'no-sequences': 'error',
'no-shadow': 'off',
'@typescript-eslint/no-shadow': 'error', // https://stackoverflow.com/questions/63961803/eslint-says-all-enums-in-typescript-app-are-already-declared-in-the-upper-scope
'no-unneeded-ternary': 'error',
'no-useless-call': 'error',
'no-useless-computed-key': 'error',
'no-useless-concat': 'error',
'no-useless-rename': 'error',
'no-var': 'error',
'operator-assignment': ['error', 'always'],
'prefer-arrow-callback': 'error',
'prefer-const': 'error',
'prefer-exponentiation-operator': 'error',
'prefer-object-spread': 'error',
'prefer-promise-reject-errors': 'error',
'prefer-regex-literals': 'error',
'prefer-rest-params': 'error',
'prefer-spread': 'error',
'prefer-template': 'error',
'spaced-comment': ['error', 'always'],
'symbol-description': 'error',
yoda: 'error',
},
overrides: [
{
files: ['./test/**/*.ts'],
rules: {
'require-atomic-updates': 'off',
},
},
],
}