Skip to content

Commit f63e6d9

Browse files
committed
Add the consistent-type-definitions rule, disabled for now
1 parent 03bac20 commit f63e6d9

File tree

1 file changed

+8
-11
lines changed

1 file changed

+8
-11
lines changed

index.js

Lines changed: 8 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -80,6 +80,14 @@ module.exports = {
8080
}
8181
],
8282
'@typescript-eslint/class-name-casing': 'error',
83+
84+
// Disabled because it's not fully usable yet:
85+
// https://github.com/typescript-eslint/typescript-eslint/issues/142
86+
// '@typescript-eslint/consistent-type-definitions': [
87+
// 'error',
88+
// 'type'
89+
// ],
90+
8391
'@typescript-eslint/explicit-function-return-type': [
8492
'error',
8593
{
@@ -183,11 +191,6 @@ module.exports = {
183191
}
184192
],
185193
'@typescript-eslint/no-triple-slash-reference': 'error',
186-
187-
// Disabled as I would like to *enforce* type aliases instead:
188-
// https://github.com/typescript-eslint/typescript-eslint/issues/142
189-
// '@typescript-eslint/no-type-alias': 'error',
190-
191194
'@typescript-eslint/no-unnecessary-qualifier': 'error',
192195
'@typescript-eslint/no-unnecessary-type-assertion': 'error',
193196
'no-unused-vars': 'off',
@@ -205,15 +208,9 @@ module.exports = {
205208
'no-useless-constructor': 'off',
206209
'@typescript-eslint/no-useless-constructor': 'error',
207210
'@typescript-eslint/no-var-requires': 'error',
208-
209211
'@typescript-eslint/prefer-for-of': 'error',
210212
'@typescript-eslint/prefer-function-type': 'error',
211213
'@typescript-eslint/prefer-includes': 'error',
212-
213-
// Disabled as I would like to *enforce* type aliases instead:
214-
// https://github.com/typescript-eslint/typescript-eslint/issues/142
215-
// '@typescript-eslint/prefer-interface': 'error',
216-
217214
'@typescript-eslint/prefer-namespace-keyword': 'error',
218215
'@typescript-eslint/prefer-string-starts-ends-with': 'error',
219216
'@typescript-eslint/promise-function-async': [

0 commit comments

Comments
 (0)