Tidy up language dropdown
This commit is contained in:
@@ -497,14 +497,14 @@ export async function findLanguage(
|
||||
}
|
||||
|
||||
// will be undefined if user cancels the quick pick.
|
||||
return await askForLanguage(cliServer, false);
|
||||
return await askForLanguage(false);
|
||||
}
|
||||
|
||||
export const supportedLanguages = ['cpp', 'csharp', 'go', 'java', 'javascript', 'python', 'ruby'];
|
||||
|
||||
export async function askForLanguage(cliServer: CodeQLCliServer, throwOnEmpty = true): Promise<string | undefined> {
|
||||
const availableLanguages = Object.keys(await cliServer.resolveLanguages()).sort();
|
||||
export async function askForLanguage(throwOnEmpty = true): Promise<string | undefined> {
|
||||
const language = await Window.showQuickPick(
|
||||
availableLanguages,
|
||||
supportedLanguages,
|
||||
{ placeHolder: 'Select target language for your query', ignoreFocusOut: true }
|
||||
);
|
||||
if (!language) {
|
||||
|
||||
@@ -134,7 +134,7 @@ async function generateQueryPack(cliServer: cli.CodeQLCliServer, queryFile: stri
|
||||
|
||||
} else {
|
||||
// open popup to ask for language if not already hardcoded
|
||||
language = fallbackLanguage || await askForLanguage(cliServer);
|
||||
language = fallbackLanguage || await askForLanguage();
|
||||
|
||||
// copy only the query file to the query pack directory
|
||||
// and generate a synthetic query pack
|
||||
|
||||
@@ -6,7 +6,7 @@ import { SemVer } from 'semver';
|
||||
import { CodeQLCliServer, QueryInfoByLanguage } from '../../cli';
|
||||
import { CodeQLExtensionInterface } from '../../extension';
|
||||
import { skipIfNoCodeQL } from '../ensureCli';
|
||||
import { getOnDiskWorkspaceFolders, getQlPackForDbscheme, languageToDbScheme } from '../../helpers';
|
||||
import { getOnDiskWorkspaceFolders, getQlPackForDbscheme, languageToDbScheme, supportedLanguages } from '../../helpers';
|
||||
import { resolveQueries } from '../../contextual/queryResolver';
|
||||
import { KeyType } from '../../contextual/keyType';
|
||||
|
||||
@@ -14,8 +14,6 @@ import { KeyType } from '../../contextual/keyType';
|
||||
* Perform proper integration tests by running the CLI
|
||||
*/
|
||||
describe('Use cli', function() {
|
||||
const supportedLanguages = ['cpp', 'csharp', 'go', 'java', 'javascript', 'python'];
|
||||
|
||||
this.timeout(60000);
|
||||
|
||||
let cli: CodeQLCliServer;
|
||||
|
||||
Reference in New Issue
Block a user