Skip to content

Commit 6ddda22

Browse files
authored
chore: move eslint rules from parserOptions to top-level rules (#3707)
1 parent 2a52874 commit 6ddda22

2 files changed

Lines changed: 16 additions & 24 deletions

File tree

.eslintrc.json

Lines changed: 15 additions & 22 deletions
Original file line numberDiff line numberDiff line change
@@ -18,27 +18,7 @@
1818
"jsx": true
1919
},
2020
"ecmaVersion": 2018,
21-
"sourceType": "module",
22-
"rules": {
23-
"curly": ["warn", "multi-line", "consistent"],
24-
"no-console": "off",
25-
"no-empty-pattern": "warn",
26-
"no-duplicate-imports": "error",
27-
"import/no-unresolved": "off",
28-
"import/export": "error",
29-
// https://github.com/typescript-eslint/typescript-eslint/blob/master/docs/getting-started/linting/FAQ.md#eslint-plugin-import
30-
// We recommend you do not use the following import/* rules, as TypeScript provides the same checks as part of standard type checking:
31-
"import/named": "off",
32-
"import/namespace": "off",
33-
"import/default": "off",
34-
"no-unused-vars": ["warn", { "argsIgnorePattern": "^_", "varsIgnorePattern": "^_" }],
35-
"@typescript-eslint/no-unused-vars": ["warn", { "argsIgnorePattern": "^_", "varsIgnorePattern": "^_" }],
36-
"@typescript-eslint/no-use-before-define": "off",
37-
"@typescript-eslint/no-empty-function": "off",
38-
"@typescript-eslint/no-empty-interface": "off",
39-
"@typescript-eslint/no-explicit-any": "off",
40-
"jest/consistent-test-it": ["error", { "fn": "it", "withinDescribe": "it" }]
41-
}
21+
"sourceType": "module"
4222
},
4323
"settings": {
4424
"react": {
@@ -68,9 +48,22 @@
6848
}
6949
],
7050
"rules": {
51+
"curly": ["warn", "multi-line", "consistent"],
52+
"no-console": "off",
53+
"no-empty-pattern": "warn",
54+
"no-duplicate-imports": "error",
7155
"import/no-unresolved": "off",
56+
"import/export": "error",
7257
"import/named": "off",
7358
"import/namespace": "off",
74-
"import/no-named-as-default-member": "off"
59+
"import/default": "off",
60+
"import/no-named-as-default-member": "off",
61+
"no-unused-vars": ["warn", { "argsIgnorePattern": "^_", "varsIgnorePattern": "^_" }],
62+
"@typescript-eslint/no-unused-vars": ["warn", { "argsIgnorePattern": "^_", "varsIgnorePattern": "^_" }],
63+
"@typescript-eslint/no-use-before-define": "off",
64+
"@typescript-eslint/no-empty-function": "off",
65+
"@typescript-eslint/no-empty-interface": "off",
66+
"@typescript-eslint/no-explicit-any": "off",
67+
"jest/consistent-test-it": ["error", { "fn": "it", "withinDescribe": "it" }]
7568
}
7669
}

packages/test-renderer/src/__tests__/RTTR.events.test.tsx

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,6 @@
11
import * as React from 'react'
22

3-
import ReactThreeTestRenderer from '../index'
4-
import type { ReactThreeTest } from '../index'
3+
import ReactThreeTestRenderer, { type ReactThreeTest } from '../index'
54

65
describe('ReactThreeTestRenderer Events', () => {
76
it('should fire an event', async () => {

0 commit comments

Comments
 (0)