Skip to content

Commit a73aafa

Browse files
isokolovskiigrevtsovna
authored andcommitted
chore(eslint): поправлена основная часть замечаний линтера и подправлены правила
1 parent 1995c94 commit a73aafa

21 files changed

Lines changed: 43 additions & 49 deletions

File tree

.prettierrc.mjs

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -25,5 +25,4 @@ const /** @type {import("prettier").Config} */ config = {
2525
],
2626
}
2727

28-
// eslint-disable-next-line import-x/no-default-export
2928
export default config

configs/eslint/rules/import.ts

Lines changed: 5 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,6 @@
11
import { defineConfig } from 'eslint/config'
22
import { createTypeScriptImportResolver } from 'eslint-import-resolver-typescript'
33

4-
// eslint-disable-next-line import-x/no-named-as-default
54
import importX, { createNodeResolver } from 'eslint-plugin-import-x'
65

76
export const importConfig = defineConfig([
@@ -28,7 +27,7 @@ export const importConfig = defineConfig([
2827
'import-x/export': 'error',
2928

3029
// https://github.com/un-ts/eslint-plugin-import-x/blob/master/docs/rules/no-deprecated.md
31-
'import-x/no-deprecated': 'warn',
30+
'import-x/no-deprecated': 'off',
3231

3332
// https://github.com/un-ts/eslint-plugin-import-x/blob/master/docs/rules/no-empty-named-blocks.md
3433
'import-x/no-empty-named-blocks': 'error',
@@ -214,13 +213,17 @@ export const importConfig = defineConfig([
214213
'**/*.d.ts',
215214
'eslint/**',
216215
'*prettier.config*',
216+
'.prettierrc*',
217+
'configs/eslint/**/*',
217218
],
218219
rules: {
219220
'import-x/no-namespace': 'off',
220221
'import-x/no-default-export': 'off',
222+
'import-x/no-named-as-default': 'off',
221223
'import-x/no-named-as-default-member': 'off',
222224
'import-x/no-anonymous-default-export': 'off',
223225
'import-x/no-nodejs-modules': 'off',
224226
},
225227
},
228+
{ files: ['index.js'], rules: { 'import-x/namespace': 'off' } },
226229
])

configs/eslint/rules/react.ts

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,6 @@
11
import { defineConfig } from 'eslint/config'
22
import react from 'eslint-plugin-react'
33

4-
// eslint-disable-next-line import-x/no-namespace
54
import * as reactHooks from 'eslint-plugin-react-hooks'
65

76
export const reactConfig = defineConfig([

configs/eslint/rules/reactNative.ts

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -48,6 +48,7 @@ export const reactNativeConfig = defineConfig([
4848
{
4949
files: ['**/*.stories.{js,jsx,cjs,mjs,ts,tsx,mts,cts}'],
5050
rules: {
51+
'react-native/no-color-literals': 'off',
5152
'react-native/no-inline-styles': 'off',
5253
'react-native/no-raw-text': 'off',
5354
},

configs/eslint/rules/typescript.ts

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -119,6 +119,9 @@ export const typescriptConfig = defineConfig([
119119
// https://typescript-eslint.io/rules/no-confusing-non-null-assertion/
120120
'@typescript-eslint/no-confusing-non-null-assertion': 'error',
121121

122+
// https://typescript-eslint.io/rules/no-deprecated/
123+
'@typescript-eslint/no-deprecated': 'warn',
124+
122125
// https://typescript-eslint.io/rules/no-dupe-class-members/
123126
'no-dupe-class-members': 'off',
124127
'@typescript-eslint/no-dupe-class-members': 'off',

index.js

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,3 @@
1-
/* eslint-disable import-x/namespace */
2-
/* eslint-disable import-x/no-deprecated */
31
import { registerRootComponent } from 'expo'
42

53
import {

src/components/Checkbox/__tests__/Checkbox.test.tsx

Lines changed: 1 addition & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -3,11 +3,7 @@ import { render } from '@testing-library/react-native'
33
import { Checkbox, type CheckboxProps } from '../Checkbox'
44

55
describe('Checkbox', () => {
6-
const defaultProps: CheckboxProps = {
7-
// eslint-disable-next-line @typescript-eslint/no-empty-function
8-
onPress: () => {},
9-
state: 'default',
10-
}
6+
const defaultProps: CheckboxProps = { onPress: jest.fn(), state: 'default' }
117

128
describe('snapshots', () => {
139
const snapshotCases: Array<[string, Partial<CheckboxProps>]> = [

src/components/Dialog/Dialog.tsx

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -24,7 +24,6 @@ export interface DialogProps extends DialogComponentProps {
2424
readonly testID?: string
2525
}
2626

27-
// eslint-disable-next-line import-x/no-deprecated
2827
const AnimatedPressable = Animated.createAnimatedComponent(Pressable)
2928

3029
export const Dialog: React.FC<DialogProps> = ({

src/components/Input/InputOtp/InputOtp.tsx

Lines changed: 3 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -91,7 +91,7 @@ export const InputOtp = memo<InputOtpProps>(
9191
)
9292

9393
const renderArray = useMemo(
94-
() => Array.from({ length }).fill(null),
94+
() => Array.from({ length }, (_, i) => `Otp-Item-${i}`),
9595
[length]
9696
)
9797

@@ -106,13 +106,12 @@ export const InputOtp = memo<InputOtpProps>(
106106
{({ pressed }) => (
107107
<>
108108
<View style={styles.content}>
109-
{renderArray.map((_, index) => (
109+
{renderArray.map((key, index) => (
110110
<InputOtpItem
111111
disabled={disabled}
112112
error={error}
113113
focused={isFocused ? index === activeIndex : false}
114-
// eslint-disable-next-line react/no-array-index-key
115-
key={index}
114+
key={key}
116115
pressed={pressed}
117116
testID={`${testID}Item`}
118117
value={value[index]}

src/components/Input/__tests__/InputGroupAddon.test.tsx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -33,6 +33,6 @@ describe('InputGroup component tests', () => {
3333

3434
fireEvent.press(pressable)
3535

36-
expect(onPress).toHaveBeenCalled()
36+
expect(onPress).toHaveBeenCalledWith()
3737
})
3838
})

0 commit comments

Comments
 (0)