diff options
| author | bors[bot] <26634292+bors[bot]@users.noreply.github.com> | 2020-02-02 21:27:58 +0000 |
|---|---|---|
| committer | GitHub <[email protected]> | 2020-02-02 21:27:58 +0000 |
| commit | ae16884b950d26546f323810170b1f8a25dc0bc0 (patch) | |
| tree | 2b51c5ea7f7cbda460ea08c535a2f56489295543 /editors/code/src/client.ts | |
| parent | 5e61c9b11def1897c4123eeafb14b4d6a5223546 (diff) | |
| parent | 2fd7af2a62ce0c8acb5daa6b2c179b638318f31a (diff) | |
Merge #2989
2989: vscode extension: migrate from any to unknown where possible r=Veetaha a=Veetaha
`unknown` type is the stricter version of `any` and it should always be prefered (like `const` over `let`).
It lets you assign any value to it, but doesn't let you carry out arbitrary operations on them without an explicit type check (like `typeof unknownValue === 'string'`).
Co-authored-by: Veetaha <[email protected]>
Diffstat (limited to 'editors/code/src/client.ts')
| -rw-r--r-- | editors/code/src/client.ts | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/editors/code/src/client.ts b/editors/code/src/client.ts index 15e1a0873..1778c4e9f 100644 --- a/editors/code/src/client.ts +++ b/editors/code/src/client.ts | |||
| @@ -68,7 +68,7 @@ PATH=${process.env.PATH} | |||
| 68 | // This also requires considering our settings strategy, which is work which needs doing | 68 | // This also requires considering our settings strategy, which is work which needs doing |
| 69 | // @ts-ignore The tracer is private to vscode-languageclient, but we need access to it to not log publishDecorations requests | 69 | // @ts-ignore The tracer is private to vscode-languageclient, but we need access to it to not log publishDecorations requests |
| 70 | res._tracer = { | 70 | res._tracer = { |
| 71 | log: (messageOrDataObject: string | any, data?: string) => { | 71 | log: (messageOrDataObject: string | unknown, data?: string) => { |
| 72 | if (typeof messageOrDataObject === 'string') { | 72 | if (typeof messageOrDataObject === 'string') { |
| 73 | if ( | 73 | if ( |
| 74 | messageOrDataObject.includes( | 74 | messageOrDataObject.includes( |