robojerk/codeql-action

fix array inclusion test

Browse source
This commit is contained in:
Robert Brignull 2020-06-01 12:45:23 +01:00
parent 28abced8ca
commit 3c2191ffdd
3 changed files with 5 additions and 5 deletions
Download patch file Download diff file Expand all files Collapse all files

4
lib/config-utils.js generated
View file

@ -60,7 +60,7 @@ class Config {
}
// Check for one of the builtin suites
if (queryUses.indexOf('/') === -1 && queryUses.indexOf('@') === -1) {
if (queryUses in builtinSuites) {
if (builtinSuites.includes(queryUses)) {
this.additionalSuites.push(queryUses);
}
else {
@ -99,7 +99,7 @@ function getQueryUsesBlank() {
}
exports.getQueryUsesBlank = getQueryUsesBlank;
function getQueryUsesIncorrect(queryUses) {
return '"uses" value for queries must be a built-in suite (' + builtinSuites.join('or') +
return '"uses" value for queries must be a built-in suite (' + builtinSuites.join(' or ') +
'), a relative path, or of the form owner/repo@ref\n' +
'Found: ' + queryUses;
}

2
lib/config-utils.js.map
View file

File diff suppressed because one or more lines are too long

4
src/config-utils.ts
View file

@ -63,7 +63,7 @@ export class Config {
// Check for one of the builtin suites
if (queryUses.indexOf('/') === -1 && queryUses.indexOf('@') === -1) {
if (queryUses in builtinSuites) {
if (builtinSuites.includes(queryUses as any)) {
this.additionalSuites.push(queryUses as BuiltInSuite);
} else {
throw new Error(getQueryUsesIncorrect(queryUses));
@ -105,7 +105,7 @@ export function getQueryUsesBlank(): string {
}
export function getQueryUsesIncorrect(queryUses: string): string {
return '"uses" value for queries must be a built-in suite (' + builtinSuites.join('or') +
return '"uses" value for queries must be a built-in suite (' + builtinSuites.join(' or ') +
'), a relative path, or of the form owner/repo@ref\n' +
'Found: ' + queryUses;
}