-
Notifications
You must be signed in to change notification settings - Fork 19
/
.eslintrc.js
93 lines (92 loc) · 4.34 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
module.exports = {
env: {
browser: true,
es2021: true,
},
extends: ['plugin:react/recommended', 'airbnb', 'prettier'],
parser: '@typescript-eslint/parser',
parserOptions: {
ecmaFeatures: {jsx: true},
ecmaVersion: 12,
sourceType: 'module',
},
plugins: ['react', '@typescript-eslint', 'react-hooks', 'unused-imports', 'prettier'],
rules: {
'no-underscore-dangle': 'off',
'react/jsx-uses-react': 'off',
'react/react-in-jsx-scope': 'off',
'react-hooks/rules-of-hooks': 'error', // Checks rules of Hooks
'no-console': 'warn',
'no-undef': 'off',
'no-unused-vars': 'off',
'import/no-extraneous-dependencies': 'off',
'prefer-destructuring': 'off',
'no-param-reassign': 'off',
'no-use-before-define': 'off',
'comma-dangle': 'off',
'no-multiple-empty-lines': 1,
'no-useless-escape': 'off',
'object-curly-newline': 'off',
'lines-between-class-members': 'off',
semi: 'warn', // should be warned and allowed for compilation
'no-restricted-globals': 'warn', // should be warned and allowed for compilation
'unused-imports/no-unused-imports-ts': 'warn',
'react/jsx-curly-brace-presence': 'warn', // should be warned and allowed for compilation
'spaced-comment': 'warn', // should be warned and allowed for compilation
'no-dupe-class-members': 'off', // false positives, when using TypeScript methods with multiple signatures
'no-useless-constructor': 'off', // false positives, when using TypeScript properties declaration in arguments
'no-empty-function': 'off', // false positives, when using TypeScript properties declaration in arguments
'react/no-unused-prop-types': 'off', // false positives, when components are wrapped with memo/forwardRef
'react/self-closing-comp': 'warn', // should be warned and allowed for compilation
'react/jsx-props-no-spreading': 0,
'react/jsx-max-props-per-line': [1, {maximum: 1, when: 'multiline'}],
// Disabled old no-shadow rule as seems to be communicated by ESLint while working with TS.
// https://github.com/typescript-eslint/typescript-eslint/blob/master/packages/eslint-plugin/docs/rules/no-shadow.md
'no-shadow': 'off',
'@typescript-eslint/no-shadow': ['error'],
'import/prefer-default-export': 'off', // cannot control what we import from standard libs
'import/no-unresolved': 'off', // typescript
'require-yield': 'off', // don't micromanage sagas or side effects
'import/extensions': 'off', // don't micromanage pretty imports
'no-unused-expressions': 'off', // prevents basic use of React exports such as in App.tsx
'react/jsx-filename-extension': [2, {extensions: ['.js', '.jsx', '.ts', '.tsx']}], // Make ESLint happy about JSX inside of tsx files
// Temporarily, we will ignore these while we introduce linting to our repo *conservatively*.
// These are to be re-enabled soon.
'arrow-body-style': 'off', // warn
'arrow-parens': 'off', // warn
'dot-notation': 'off', // required for our env variables currently
'prefer-const': 'off',
'max-len': 'off',
'react/jsx-no-target-blank': 'off', // target="_blank" without rel="noreferrer" is a security risk: see https://html.spec.whatwg.org/multipage/links.html#link-type-noopener react/jsx-no-target-blank
'react/prop-types': 'off', // TODO re-enable
'react/require-default-props': 'off', // TODO re-enable
'no-nested-ternary': 'off', // warn
'no-redeclare': 'off',
'@typescript-eslint/no-redeclare': 'error',
'consistent-return': 'off', // warn. Look at api calls closely before enabling this. api.ts.
// Accessibility off for now to make speed a priority and avoid restructuring for now
'jsx-a11y/click-events-have-key-events': 'off',
'jsx-a11y/no-static-element-interactions': 'off',
'jsx-a11y/anchor-is-valid': 'off', // TODO warn because, we should really be using buttons.
'jsx-a11y/no-noninteractive-element-interactions': 'off',
'jsx-a11y/alt-text': 'off',
'space-in-brackets': 'off',
'jsx-a11y/label-has-associated-control': [
'warn',
{
controlComponents: ['Select'],
assert: 'either',
depth: 3,
},
],
'no-implicit-coercion': [
'error',
{
boolean: true,
number: true,
string: true,
},
],
},
ignorePatterns: ['package.json', './node_modules', './dist', '**/dist/*.js'],
};