Skip to content

Commit

Permalink
Apply suggestions from code review
Browse files Browse the repository at this point in the history
Co-authored-by: Andrew Eisenberg <aeisenberg@github.com>
  • Loading branch information
edoardopirovano and aeisenberg authored Mar 16, 2021
1 parent c267dd4 commit cd35766
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion extensions/ql-vscode/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -186,7 +186,7 @@
"scope": "window",
"type": "string",
"default": "",
"description": "Additional command line arguments to pass to the CLI when running tests."
"markdownDescription": "Additional command line arguments to pass to the CLI when [running tests](https://codeql.github.com/docs/codeql-cli/manual/test-run/)."
},
"codeQL.runningTests.numberOfThreads": {
"scope": "window",
Expand Down
2 changes: 1 addition & 1 deletion extensions/ql-vscode/src/cli.ts
Original file line number Diff line number Diff line change
Expand Up @@ -509,7 +509,7 @@ export class CodeQLCliServer implements Disposable {
testPaths: string[], workspaces: string[], options: TestRunOptions
): AsyncGenerator<TestCompleted, void, unknown> {

const subcommandArgs = this.cliConfig.additionalTestArguments.toString().split(' ').concat([
const subcommandArgs = this.cliConfig.additionalTestArguments.toString().split(/\s+/).concat([
'--additional-packs', workspaces.join(path.delimiter),
'--threads',
this.cliConfig.numberTestThreads.toString(),
Expand Down

0 comments on commit cd35766

Please sign in to comment.