Skip to content

Commit b380085

Browse files
linting changes
1 parent b70c885 commit b380085

File tree

73 files changed

+2714
-4402
lines changed

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

73 files changed

+2714
-4402
lines changed

.eslintignore

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -2,5 +2,3 @@
22
lib
33
test/disabled
44
!etc/docs
5-
6-
test/explicit-resource-management

.eslintrc.json

Lines changed: 33 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -6,13 +6,11 @@
66
},
77
"plugins": [
88
"simple-import-sort",
9-
"import",
109
"@typescript-eslint",
1110
"prettier",
1211
"unused-imports",
1312
"tsdoc",
14-
"mocha",
15-
"github"
13+
"mocha"
1614
],
1715
"extends": [
1816
"eslint:recommended",
@@ -67,15 +65,17 @@
6765
],
6866
"simple-import-sort/imports": "error",
6967
"simple-import-sort/exports": "error",
70-
"import/first": "error",
71-
"import/newline-after-import": "error",
72-
"import/no-duplicates": "error",
7368
"strict": [
7469
"error",
7570
"global"
7671
],
7772
"@typescript-eslint/no-explicit-any": "off",
73+
"@typescript-eslint/no-non-null-assertion": "error",
74+
"@typescript-eslint/no-redundant-type-constituents": "off",
75+
"@typescript-eslint/no-empty-function": "error",
76+
"@typescript-eslint/no-require-imports": "off",
7877
"@typescript-eslint/require-await": "off",
78+
"@typescript-eslint/no-unused-expressions": "off",
7979
"@typescript-eslint/consistent-type-imports": [
8080
"error",
8181
{
@@ -121,12 +121,7 @@
121121
"message": "Do not strictly check typeof undefined (NOTE: currently this rule only detects the usage of 'undefined' string literal so this could be a misfire)"
122122
}
123123
],
124-
"@typescript-eslint/no-unused-vars": [
125-
"error",
126-
{
127-
"argsIgnorePattern": "^_"
128-
}
129-
]
124+
"@typescript-eslint/no-unused-vars": "error"
130125
},
131126
"overrides": [
132127
{
@@ -168,6 +163,7 @@
168163
"@typescript-eslint/no-empty-function": "off",
169164
"@typescript-eslint/no-this-alias": "off",
170165
"@typescript-eslint/no-unused-vars": "off",
166+
"@typescript-eslint/consistent-type-imports": "off",
171167
// Since we use ts-node we should always require the TS code
172168
// Also we have patches to apply to the mock server, so we should always import from there
173169
"no-restricted-modules": [
@@ -200,6 +196,15 @@
200196
"no-restricted-syntax": "off",
201197
"typescript-eslint/ban-ts-comment": "off",
202198
"no-restricted-imports": "off",
199+
"@typescript-eslint/no-unused-vars": [
200+
"error",
201+
{
202+
"argsIgnorePattern": "^_",
203+
"caughtErrorsIgnorePattern": "^_",
204+
"destructuredArrayIgnorePattern": "^_",
205+
"varsIgnorePattern": "^_"
206+
}
207+
],
203208
"@typescript-eslint/no-restricted-imports": [
204209
"error",
205210
{
@@ -235,12 +240,23 @@
235240
"@typescript-eslint/restrict-plus-operands": "off",
236241
"@typescript-eslint/restrict-template-expressions": "off",
237242
"@typescript-eslint/require-await": "off",
243+
"@typescript-eslint/no-redundant-type-constituents": "off",
244+
"@typescript-eslint/no-unused-expressions": "off",
245+
"@typescript-eslint/no-explicit-any": "off",
238246
"no-return-await": "off",
239247
"@typescript-eslint/return-await": [
240248
"error",
241249
"always"
242250
],
243-
"github/no-then": "error",
251+
"@typescript-eslint/no-unused-vars": [
252+
"error",
253+
{
254+
"argsIgnorePattern": "^_",
255+
"caughtErrorsIgnorePattern": "^_",
256+
"destructuredArrayIgnorePattern": "^_",
257+
"varsIgnorePattern": "^_"
258+
}
259+
],
244260
"no-restricted-imports": [
245261
"error",
246262
{
@@ -263,7 +279,7 @@
263279
"tsdoc/syntax": "warn",
264280
"no-console": "off",
265281
"@typescript-eslint/no-explicit-any": "off",
266-
"@typescript-eslint/no-unused-vars": "error",
282+
"@typescript-eslint/no-unused-vars": "off",
267283
"@typescript-eslint/ban-ts-comment": "off",
268284
"@typescript-eslint/no-empty-function": "off"
269285
}
@@ -276,7 +292,9 @@
276292
],
277293
"parser": "@typescript-eslint/parser",
278294
"rules": {
279-
"unused-imports/no-unused-imports": "error"
295+
"unused-imports/no-unused-imports": "error",
296+
"@typescript-eslint/no-empty-object-type": "off",
297+
"@typescript-eslint/no-unsafe-declaration-merging": "off"
280298
}
281299
}
282300
]

0 commit comments

Comments
 (0)