-
Notifications
You must be signed in to change notification settings - Fork 1
/
.eslintrc.cjs
85 lines (84 loc) · 2.09 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
const vitestFiles = ['app/**/__tests__/**/*', 'app/**/*.{spec,test}.*']
const testFiles = ['**/tests/**', ...vitestFiles]
const appFiles = ['app/**']
/** @type {import('@types/eslint').Linter.BaseConfig} */
module.exports = {
extends: [
'@remix-run/eslint-config',
'@remix-run/eslint-config/node',
'prettier',
],
rules: {
// playwright requires destructuring in fixtures even if you don't use anything 🤷♂️
'no-empty-pattern': 'off',
'@typescript-eslint/consistent-type-imports': [
'warn',
{
prefer: 'type-imports',
disallowTypeAnnotations: true,
fixStyle: 'inline-type-imports',
},
],
'import/no-duplicates': ['warn', { 'prefer-inline': true }],
'import/consistent-type-specifier-style': ['warn', 'prefer-inline'],
'import/order': [
'warn',
{
alphabetize: { order: 'asc', caseInsensitive: true },
groups: [
'builtin',
'external',
'internal',
'parent',
'sibling',
'index',
],
},
],
},
overrides: [
{
plugins: ['remix-react-routes'],
files: appFiles,
excludedFiles: testFiles,
rules: {
'remix-react-routes/use-link-for-routes': 'error',
'remix-react-routes/require-valid-paths': 'error',
// disable this one because it doesn't appear to work with our
// route convention. Someone should dig deeper into this...
'remix-react-routes/no-relative-paths': [
'off',
{ allowLinksToSelf: true },
],
'remix-react-routes/no-urls': 'error',
'no-restricted-imports': [
'error',
{
patterns: [
{
group: testFiles,
message: 'Do not import test files in app files',
},
],
},
],
},
},
{
extends: ['@remix-run/eslint-config/jest-testing-library'],
files: vitestFiles,
rules: {
'testing-library/no-await-sync-events': 'off',
'jest-dom/prefer-in-document': 'off',
},
// we're using vitest which has a very similar API to jest
// (so the linting plugins work nicely), but it means we have to explicitly
// set the jest version.
settings: {
jest: {
version: 28,
},
},
},
],
}