Skip to content

Commit 750145d

Browse files
authored
ci(amazonq): fix linting issue on language server child process (#6332)
## Problem In a recent PR we disabled using child_process in favour of our internal one. This started flagging lsp/client.ts, since it wasn't skipped. ## Solution Skip eslint for child_process in this case, since the language server options expect the actual child process. The same is done for the workspace context language server: https://github.com/aws/aws-toolkit-vscode/blob/master/packages/core/src/amazonq/lsp/lspClient.ts#L13 --- - Treat all work as PUBLIC. Private `feature/x` branches will not be squash-merged at release time. - Your code changes must meet the guidelines in [CONTRIBUTING.md](https://github.com/aws/aws-toolkit-vscode/blob/master/CONTRIBUTING.md#guidelines). - License: I confirm that my contribution is made under the terms of the Apache 2.0 license.
1 parent 35b679c commit 750145d

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

packages/amazonq/src/lsp/client.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,7 @@
55

66
import vscode, { env, version } from 'vscode'
77
import * as nls from 'vscode-nls'
8-
import * as cp from 'child_process'
8+
import * as cp from 'child_process' // eslint-disable-line no-restricted-imports -- language server options expect actual child process
99
import * as crypto from 'crypto'
1010
import { LanguageClient, LanguageClientOptions, ServerOptions, TransportKind } from 'vscode-languageclient'
1111
import { registerInlineCompletion } from '../inline/completion'

0 commit comments

Comments
 (0)