/
.eslintrc.cjs
133 lines (128 loc) · 5.73 KB
/
.eslintrc.cjs
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
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
const eslintRules = {
'arrow-body-style': 'warn', // WARN b/c blocks style allows for readability and ensure scope
'arrow-spacing': 'error',
'eol-last': 'error',
eqeqeq: 'error',
'func-call-spacing': 'error',
indent: 'off', // OFF b/c causes problems between Prettier and ESLint
'linebreak-style': 'off', // OFF b/c Windows (Git) puts CRLF line endings
'multiline-ternary': 'off', // OFF b/c causes problems between Prettier and ESLint
'no-alert': 'error',
'no-async-promise-executor': 'error',
'no-case-declarations': 'error',
'no-console': ['error', { allow: ['error', 'warn'] }],
'no-control-regex': 'error',
'no-dupe-keys': 'error',
'no-empty': 'error',
'no-extra-boolean-cast': 'error',
'no-extra-parens': 'off', // OFF b/c reactive dependencies in .svelte files (mostly for errors)
'no-extra-semi': 'error',
'no-fallthrough': 'error',
'no-import-assign': 'error',
'no-irregular-whitespace': 'error',
'no-prototype-builtins': 'error',
'no-return-await': 'error',
'no-trailing-spaces': 'error',
'no-useless-escape': 'error',
'no-undef': 'error',
'no-underscore-dangle': 'off', // OFF b/c this syntax is used for defining local callback methods
'no-unreachable': 'error',
'no-unused-vars': 'off', // OFF b/c the typescript linter rule is used
'no-var': 'error',
'prefer-arrow-callback': 'warn',
'prefer-const': 'warn',
'prefer-destructuring': 'off', // OFF b/c it's not really correct
quotes: ['error', 'single'],
semi: 'off', // OFF b/c we aren't using semicolons
'space-before-function-paren': 'off', // OFF b/c we aren't using spaces before function parameters / signatures
'spaced-comment': 'error',
}
const eslintRulesOnlyTypescript = {
'no-undef': 'off', // Typescript handles undefined variables better than eslint
}
const typescriptEslintRules = {
'@typescript-eslint/array-type': 'error',
'@typescript-eslint/await-thenable': 'error',
'@typescript-eslint/ban-types': 'error',
'@typescript-eslint/ban-ts-comment': 'warn',
'@typescript-eslint/explicit-function-return-type': 'warn', // Warn b/c return types are not supported in Svelte markdown
'@typescript-eslint/explicit-module-boundary-types': 'error',
'@typescript-eslint/no-array-constructor': 'error',
'@typescript-eslint/no-empty-function': 'off', // OFF b/c we use empty functions a lot (esp. for initialization)
'@typescript-eslint/no-explicit-any': 'error',
'@typescript-eslint/no-extra-non-null-assertion': 'error',
'@typescript-eslint/no-extra-semi': 'error',
'@typescript-eslint/no-floating-promises': 'warn', // Warn b/c we have existing code in migration that I don't want to touch to pass new linting rules
'@typescript-eslint/no-implied-eval': 'error',
'@typescript-eslint/no-inferrable-types': 'off', // OFF b/c this errors on some useful code annotations for function signatures
'@typescript-eslint/no-misused-new': 'error',
'@typescript-eslint/no-misused-promises': 'error',
'@typescript-eslint/no-non-null-asserted-optional-chain': 'error',
'@typescript-eslint/no-non-null-assertion': 'error',
'@typescript-eslint/no-this-alias': 'error',
'@typescript-eslint/no-unnecessary-type-assertion': 'error',
'@typescript-eslint/no-unsafe-assignment': 'off', // OFF b/c used in Svelte components for UI logic
'@typescript-eslint/no-unsafe-call': 'off', // OFF b/c used in Svelte components for UI logic
'@typescript-eslint/no-unsafe-member-access': 'off', // OFF b/c there are simply too many linting errors
'@typescript-eslint/no-unsafe-return': 'off', // OFF b/c used in Svelte components for UI logic
'@typescript-eslint/no-unsafe-argument': 'off', // OFF b/c ESlint resolves types of the absolute imports as any
'@typescript-eslint/no-unused-vars': 'error',
'@typescript-eslint/no-var-requires': 'error',
'@typescript-eslint/prefer-regexp-exec': 'error',
'@typescript-eslint/restrict-plus-operands': 'off', // OFF b/c not entirely accurate despite proper typings
'@typescript-eslint/restrict-template-expressions': 'off', // OFF b/c using any is useful in template expressions
'@typescript-eslint/require-await': 'error',
'@typescript-eslint/unbound-method': 'error',
'@typescript-eslint/no-namespace': 'off', // OFF b/c used in Svelte components for exporting types inside of a namespace
}
const linterRules = {
...eslintRules,
...eslintRulesOnlyTypescript,
...typescriptEslintRules,
}
const svelteRules = {
'@typescript-eslint/no-explicit-any': 'off', // OFF b/c used for callback methods in Svelte components
'svelte/no-at-html-tags': 'off', // OFF b/c we use HTML tags in markdown sections
'svelte/block-lang': [
'warn',
{
script: ['ts', null],
style: ['postcss'], // enforce postcss
},
],
}
module.exports = {
root: true,
extends: [
'eslint:recommended',
'plugin:@typescript-eslint/recommended',
'plugin:svelte/recommended',
'prettier',
],
parser: '@typescript-eslint/parser',
plugins: ['@typescript-eslint'],
parserOptions: {
sourceType: 'module',
project: './tsconfig.lint.json',
ecmaVersion: 2020,
extraFileExtensions: ['.svelte', '.html'],
},
env: {
browser: true,
es2017: true,
node: true,
},
rules: { ...eslintRules },
ignorePatterns: ['src/app.html'],
overrides: [
{
files: ['*.svelte'],
parser: 'svelte-eslint-parser',
rules: { ...linterRules, ...svelteRules },
parserOptions: {
project: './tsconfig.lint.json',
parser: '@typescript-eslint/parser',
},
},
],
}