Skip to content

Commit 39e6b27

Browse files
committed
Make linter happy
1 parent d8435d1 commit 39e6b27

3 files changed

Lines changed: 7 additions & 6 deletions

File tree

extensions/ql-vscode/src/cli.ts

Lines changed: 5 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1149,7 +1149,7 @@ export class CodeQLCliServer implements Disposable {
11491149
const args = this.getAdditionalPacksArg(additionalPacks);
11501150
if (extensionPacksOnly) {
11511151
if (!(await this.cliConstraints.supportsQlpacksKind())) {
1152-
this.logger.log(
1152+
void this.logger.log(
11531153
"Warning: Running with extension packs is only supported by CodeQL CLI v2.12.3 or later.",
11541154
);
11551155
return {};
@@ -1160,8 +1160,9 @@ export class CodeQLCliServer implements Disposable {
11601160
return this.runJsonCodeQlCliCommand<QlpacksInfo>(
11611161
["resolve", "qlpacks"],
11621162
args,
1163-
"Resolving qlpack information" +
1164-
(extensionPacksOnly ? " (extension packs only)" : ""),
1163+
`Resolving qlpack information${
1164+
extensionPacksOnly ? " (extension packs only)" : ""
1165+
}`,
11651166
);
11661167
}
11671168

@@ -1348,7 +1349,7 @@ export class CodeQLCliServer implements Disposable {
13481349
}
13491350

13501351
public async setUseExtensionPacks(useExtensionPacks: boolean) {
1351-
this.cliConfig.setUseExtensionPacks(useExtensionPacks);
1352+
await this.cliConfig.setUseExtensionPacks(useExtensionPacks);
13521353
}
13531354
}
13541355

extensions/ql-vscode/test/vscode-tests/cli-integration/queries.test.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -99,7 +99,7 @@ describeWithCodeQL()("Queries", () => {
9999
});
100100

101101
describe("extension packs", () => {
102-
skipIfTrue(qs.cliServer.cliConstraints.supportsQlpacksKind());
102+
skipIfTrue(qs.cliServer.cliConstraints.supportsQlpacksKind);
103103

104104
const queryUsingExtensionPath = join(
105105
__dirname,

extensions/ql-vscode/test/vscode-tests/cli.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -43,7 +43,7 @@ export function itWithCodeQL() {
4343
return it;
4444
}
4545

46-
export async function skipIfTrue(condition: Thenable<boolean>) {
46+
export function skipIfTrue(condition: () => Thenable<boolean>) {
4747
return beforeEach(async () => {
4848
if (await condition) {
4949
pending();

0 commit comments

Comments
 (0)