bump dependency versions, fix new lint guidance

Author: bollwyvl

.gitignore

@@ -113,6 +113,6 @@ atest/output/
 junit.xml
 coverage/
 .vscode/
-_schema.d.ts
+_*.d.ts
 _build
 .virtual_documents/

atest/07_Configuration.robot

@@ -14,7 +14,6 @@ Python
 
 YAML
     [Documentation]    EXPECT FAIL Composer YAML files don't allow a "greetings" key
-    [Tags]    expect:fail
     Settings Should Change Editor Diagnostics    YAML    example.yaml    yaml-language-server
     ...    {"yaml.schemas": {"http://json.schemastore.org/composer": "*"}}
     ...    duplicate key

atest/ports.py

@@ -4,9 +4,9 @@
 
 
 def get_unused_port():
-    """ Get an unused port by trying to listen to any random port.
+    """Get an unused port by trying to listen to any random port.
 
-        Probably could introduce race conditions if inside a tight loop.
+    Probably could introduce race conditions if inside a tight loop.
     """
     sock = socket.socket(socket.AF_INET, socket.SOCK_STREAM)
     sock.bind(("localhost", 0))

docs/Language Servers.ipynb

@@ -42,6 +42,7 @@
     "\n",
     "import IPython\n",
     "from jinja2 import Template\n",
+    "\n",
     "from jupyter_lsp import LanguageServerManager"
    ]
   },

docs/conf.py

@@ -102,7 +102,6 @@
 
 
 def setup(app):
-    """ Runs before the "normal business" of sphinx. Don't go too crazy here.
-    """
+    """Runs before the "normal business" of sphinx. Don't go too crazy here."""
     app.add_css_file("css/custom.css")
     subprocess.check_call(["jlpm", "--ignore-optional"])

package.json

@@ -1,12 +1,12 @@
 {
   "dependencies": {},
   "devDependencies": {
-    "@typescript-eslint/eslint-plugin": "^2.27.0",
-    "@typescript-eslint/parser": "^2.27.0",
+    "@typescript-eslint/eslint-plugin": "^3.10.1",
+    "@typescript-eslint/parser": "^3.10.1",
     "bash-language-server": "^1.6.1",
     "dictionary-en": "^3.0.0",
     "dockerfile-language-server-nodejs": "^0.0.22",
-    "eslint": "^6.8.0",
+    "eslint": "^7.5.0",
     "eslint-config-prettier": "^6.7.0",
     "eslint-plugin-import": "^2.22.0",
     "eslint-plugin-jest": "^23.8.2",
@@ -17,14 +17,14 @@
     "jest-github-actions-reporter": "^1.0.2",
     "lerna": "^3.13.2",
     "precise-commits": "^1.0.2",
-    "prettier": "^2.0.5",
+    "prettier": "^2.1.2",
     "sql-language-server": "^0.11.4",
-    "typescript": "~3.9.5",
+    "typescript": "~4.0.2",
     "unified-language-server": "^0.3.0",
     "vscode-css-languageserver-bin": "^1.4.0",
     "vscode-html-languageserver-bin": "^1.4.0",
     "vscode-json-languageserver-bin": "^1.0.1",
-    "yaml-language-server": "~0.4.0"
+    "yaml-language-server": "^0.11.1"
   },
   "husky": {
     "hooks": {}

packages/.eslintrc.js

@@ -21,21 +21,18 @@ module.exports = {
     '**/_*.ts',
     '**/_*.d.ts',
     '**/typings/**/*.d.ts',
-    '**/dist/*'
+    '**/dist/*',
+    'packages/.eslintrc.js'
   ],
   parser: '@typescript-eslint/parser',
   parserOptions: {
     project: 'packages/tsconfig.eslint.json'
   },
   plugins: ['@typescript-eslint', 'jest', 'import'],
   rules: {
-    '@typescript-eslint/ban-ts-ignore': 'warn',
     '@typescript-eslint/camelcase': 'off',
     '@typescript-eslint/explicit-function-return-type': 'off',
-    '@typescript-eslint/interface-name-prefix': [
-      'error',
-      { prefixWithI: 'always' }
-    ],
+    '@typescript-eslint/explicit-module-boundary-types': 'off',
     '@typescript-eslint/no-empty-interface': 'off',
     '@typescript-eslint/no-explicit-any': 'off',
     '@typescript-eslint/no-floating-promises': ['error', { ignoreVoid: true }],

packages/jupyterlab-go-to-definition/src/editors/codemirror/extension.ts

@@ -32,7 +32,8 @@ function getModifierState(event: MouseEvent, modifierKey: string): boolean {
   }
 }
 
-export class CodeMirrorExtension extends CodeMirrorTokensProvider
+export class CodeMirrorExtension
+  extends CodeMirrorTokensProvider
   implements IEditorExtension {
   jumper: CodeJumper;
   static modifierKey: KeyModifier;

packages/jupyterlab-go-to-definition/src/jumpers/jumper.ts

@@ -358,7 +358,7 @@ export abstract class CodeJumper {
     }
   }
 
-  handle_kernel_inspect(response: IIOPubMessage, fallback: Function) {
+  handle_kernel_inspect(response: IIOPubMessage, fallback: () => void) {
     let obj: any = response.content;
     if (obj.name === 'stdout') {
       let data = JSON.parse(obj.text);
@@ -386,8 +386,8 @@ export abstract class CodeJumper {
   protected inspect_and_jump(
     context: TokenContext,
     cell_of_origin_analyzer: LanguageAnalyzer,
-    fallback: Function,
-    callback: Function
+    fallback: () => void,
+    callback: () => void
   ) {
     let code = cell_of_origin_analyzer.definitionLocationQuery(context);
 

packages/jupyterlab-go-to-definition/src/languages/python.ts

@@ -6,7 +6,7 @@ import { PathExt } from '@jupyterlab/coreutils';
 function evaluateSkippingBrackets(
   tokens: ReadonlyArray<IToken>,
   indexShift: number,
-  callback: Function,
+  callback: (nextToken: CodeEditor.IToken, indexShift: number) => any,
   allowNegativeBrackets = false
 ) {
   // here `nextToken` is any token, not necessarily a meaningful one