Skip to content

Commit 4e15662

Browse files
committed
chore: removed unused rules
1 parent 199d5eb commit 4e15662

File tree

1 file changed

+6
-17
lines changed

1 file changed

+6
-17
lines changed

.eslintrc.js

Lines changed: 6 additions & 17 deletions
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,6 @@ module.exports = {
77
node: true,
88
},
99
extends: [
10-
// 'plugin:@typescript-eslint/eslint-recommended',
1110
'plugin:@typescript-eslint/recommended',
1211
'plugin:prettier/recommended',
1312
],
@@ -23,54 +22,44 @@ module.exports = {
2322
},
2423
},
2524
rules: {
26-
// '@typescript-eslint/ban-ts-ignore': ['off'],
27-
// '@typescript-eslint/camelcase': ['off'],
2825
'@typescript-eslint/explicit-function-return-type': [
2926
'error',
3027
{ allowExpressions: true },
3128
], // Enforce return type definitions for functions
3229
'@typescript-eslint/explicit-member-accessibility': 'error', // Enforce explicit accessibility modifiers on class properties and methods (public, private, protected)
33-
// '@typescript-eslint/indent': [ 'error', 2, { SwitchCase: 1 } ], // Rule is already defined in eslint core, also not recommended by typescript-eslint - prettier could be used instead
34-
// '@typescript-eslint/interface-name-prefix': ['off'], // Probably not needed
35-
// '@typescript-eslint/member-delimiter-style': [ 'error', { multiline: { delimiter: 'none' } } ], // Propose to disable this rule in favor of prettier as recommended here https://typescript-eslint.io/rules/member-delimiter-style/
3630
'@typescript-eslint/member-ordering': [
3731
// Standardize the order of class members
3832
'error',
3933
{
4034
default: {
4135
memberTypes: [
4236
'signature',
43-
'public-field', // = ["public-static-field", "public-instance-field"]
44-
'protected-field', // = ["protected-static-field", "protected-instance-field"]
45-
'private-field', // = ["private-static-field", "private-instance-field"]
37+
'public-field',
38+
'protected-field',
39+
'private-field',
4640
'constructor',
47-
'public-method', // = ["public-static-method", "public-instance-method"]
48-
'protected-method', // = ["protected-static-method", "protected-instance-method"]
49-
'private-method', // = ["private-static-method", "private-instance-method"]
41+
'public-method',
42+
'protected-method',
43+
'private-method',
5044
],
5145
order: 'alphabetically',
5246
},
5347
},
5448
],
5549
'@typescript-eslint/no-explicit-any': 'error', // Disallow usage of the any type
56-
// '@typescript-eslint/no-inferrable-types': ['off'], // Propose to disable this rule - read here why https://typescript-eslint.io/rules/no-inferrable-types/
5750
'@typescript-eslint/no-unused-vars': ['error', { argsIgnorePattern: '^_' }], // Disallow unused variables, except for variables starting with an underscore
5851
'@typescript-eslint/no-use-before-define': ['off'], // Check if this rule is needed
59-
// '@typescript-eslint/semi': [ 'error', 'always' ], Rule is already defined in eslint core, and also not recommended by typescript-eslint - prettier could be used instead
6052
'no-unused-vars': 'off', // Disable eslint core rule, since it's replaced by @typescript-eslint/no-unused-vars
6153
// Rules from eslint core https://eslint.org/docs/latest/rules/
6254
'array-bracket-spacing': ['error', 'never'], // Disallow spaces inside of array brackets
63-
// 'arrow-body-style': [ 'error', 'as-needed' ], Default is already 'as-needed'
6455
'computed-property-spacing': ['error', 'never'], // Disallow spaces inside of computed properties
6556
'func-style': ['warn', 'expression'], // Enforce function expressions instead of function declarations
66-
// indent: ['error', 2, { SwitchCase: 1 }],
6757
'keyword-spacing': 'error', // Enforce spaces after keywords and before parenthesis, e.g. if (condition) instead of if(condition)
6858
'padding-line-between-statements': [
6959
// Require an empty line before return statements
7060
'error',
7161
{ blankLine: 'always', prev: '*', next: 'return' },
7262
],
73-
// 'newline-before-return': 2, // Deprecated in favor of padding-line-between-statements
7463
'no-console': 0, // Allow console.log statements
7564
'no-multi-spaces': ['error', { ignoreEOLComments: false }], // Disallow multiple spaces except for comments
7665
'no-multiple-empty-lines': ['error', { max: 1, maxBOF: 0, maxEOF: 0 }], // Enforce no empty line at the beginning & end of files and max 1 empty line between consecutive statements

0 commit comments

Comments
 (0)