Skip to content

Commit b47bfd6

Browse files
committed
restore interface name eslint rule, grandfather in VirtualEditor
1 parent e9b897f commit b47bfd6

File tree

2 files changed

+21
-17
lines changed

2 files changed

+21
-17
lines changed

packages/.eslintrc.js

Lines changed: 20 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -20,42 +20,46 @@ module.exports = {
2020
'**/lib/**/*',
2121
'**/_*.ts',
2222
'**/_*.d.ts',
23-
'**/typings/**/*.d.ts'
23+
'**/typings/**/*.d.ts',
24+
'**/dist/*'
2425
],
2526
parser: '@typescript-eslint/parser',
2627
parserOptions: {
2728
project: 'packages/tsconfig.eslint.json'
2829
},
2930
plugins: ['@typescript-eslint', 'jest'],
3031
rules: {
31-
'@typescript-eslint/no-floating-promises': ['error', { ignoreVoid: true }],
32-
'@typescript-eslint/no-unused-vars': ['warn', { args: 'none' }],
33-
'@typescript-eslint/no-use-before-define': 'off',
32+
'@typescript-eslint/ban-ts-ignore': 'warn',
3433
'@typescript-eslint/camelcase': 'off',
35-
'@typescript-eslint/no-explicit-any': 'off',
36-
'@typescript-eslint/no-non-null-assertion': 'off',
37-
'@typescript-eslint/no-namespace': 'off',
3834
'@typescript-eslint/explicit-function-return-type': 'off',
39-
'@typescript-eslint/ban-ts-ignore': 'warn',
40-
'@typescript-eslint/no-var-requires': 'off',
35+
'@typescript-eslint/interface-name-prefix': [
36+
'error',
37+
{ prefixWithI: 'always' }
38+
],
4139
'@typescript-eslint/no-empty-interface': 'off',
40+
'@typescript-eslint/no-explicit-any': 'off',
41+
'@typescript-eslint/no-floating-promises': ['error', { ignoreVoid: true }],
4242
'@typescript-eslint/no-inferrable-types': 'off',
43+
'@typescript-eslint/no-namespace': 'off',
44+
'@typescript-eslint/no-non-null-assertion': 'off',
45+
'@typescript-eslint/no-unused-vars': ['warn', { args: 'none' }],
46+
'@typescript-eslint/no-use-before-define': 'off',
47+
'@typescript-eslint/no-var-requires': 'off',
48+
'jest/expect-expect': 'off',
49+
'jest/no-export': 'warn',
50+
'jest/no-jest-import': 'off',
51+
'jest/no-try-expect': 'warn',
52+
'no-case-declarations': 'warn',
53+
'no-control-regex': 'warn',
4354
'no-inner-declarations': 'off',
4455
'no-prototype-builtins': 'off',
45-
'no-control-regex': 'warn',
4656
'no-undef': 'warn',
47-
'no-case-declarations': 'warn',
4857
'no-useless-escape': 'off',
4958
'prefer-const': 'off',
50-
'jest/no-jest-import': 'off',
51-
'jest/no-export': 'warn',
52-
'jest/no-try-expect': 'warn',
53-
'jest/expect-expect': 'off',
5459
// deviations from jupyterlab, should probably be fixed
5560
'jest/valid-expect': 'off',
5661
'jest/no-test-callback': 'off',
5762
'@typescript-eslint/ban-ts-ignore': 'off',
58-
'@typescript-eslint/interface-name-prefix': 'off',
5963
'react/display-name': 'off',
6064
'prefer-spread': 'off',
6165
'@typescript-eslint/triple-slash-reference': 'off',

packages/jupyterlab-lsp/src/virtual/editor.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -253,4 +253,4 @@ export abstract class VirtualEditor implements CodeMirror.Editor {
253253
}
254254
}
255255

256-
export interface VirtualEditor extends CodeMirror.Editor {}
256+
export interface VirtualEditor extends CodeMirror.Editor {} // eslint-disable-line @typescript-eslint/interface-name-prefix

0 commit comments

Comments
 (0)