Skip to content

Commit 374edb4

Browse files
committed
fixup
1 parent 1bb9691 commit 374edb4

File tree

3 files changed

+3
-11
lines changed

3 files changed

+3
-11
lines changed

packages/amazonq/src/lsp/activation.ts

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,6 @@ import path from 'path'
88
import { AmazonQLSPDownloader } from './download'
99

1010
export async function activate(ctx: vscode.ExtensionContext): Promise<void> {
11-
// use the codewhisperer language server
1211
const serverPath = ctx.asAbsolutePath(path.join('resources', 'qdeveloperserver'))
1312
const clientPath = ctx.asAbsolutePath(path.join('resources', 'qdeveloperclient'))
1413
await new AmazonQLSPDownloader(serverPath, clientPath).tryInstallLsp()

packages/amazonq/src/lsp/download.ts

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -51,23 +51,22 @@ export class AmazonQLSPDownloader extends LSPDownloader {
5151
try {
5252
tempFolder = await makeTemporaryToolkitFolder()
5353

54-
// the business logic
54+
// download and extract the business logic
5555
await this.downloadAndExtractServer({
5656
content: server,
5757
installLocation: this.serverPath,
5858
name: 'qdeveloperserver',
5959
tempFolder,
6060
})
6161

62-
// mynah ui
62+
// download and extract mynah ui
6363
await this.downloadAndExtractServer({
6464
content: clients,
6565
installLocation: this.clientPath,
6666
name: 'qdeveloperclient',
6767
tempFolder,
6868
})
6969
} finally {
70-
// clean up temp folder
7170
if (tempFolder) {
7271
await tryRemoveFolder(tempFolder)
7372
}

packages/core/src/shared/downloadLSP.ts

Lines changed: 1 addition & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -139,9 +139,6 @@ export abstract class LSPDownloader {
139139
return undefined
140140
}
141141

142-
/**
143-
* Downloads servers.zip, clients.zip, qserver.zip and then extracts them
144-
*/
145142
async downloadAndExtractServer({
146143
content,
147144
installLocation,
@@ -168,9 +165,6 @@ export abstract class LSPDownloader {
168165
await fs.rename(path.join(tempFolder, name), installLocation)
169166
}
170167

171-
/**
172-
* Install a runtime from the manifest to the runtime location
173-
*/
174168
async installRuntime(runtime: Content, installLocation: string, tempPath: string) {
175169
const downloadNodeOk = await this.downloadAndCheckHash(tempPath, runtime)
176170
if (!downloadNodeOk) {
@@ -191,7 +185,7 @@ export abstract class LSPDownloader {
191185
abstract cleanup(): Promise<boolean>
192186

193187
/**
194-
* Given a manifest install any servers and runtimes that are required to disk
188+
* Given a manifest install any servers and runtimes that are required
195189
*/
196190
abstract install(manifest: Manifest): Promise<boolean>
197191

0 commit comments

Comments
 (0)