robojerk/codeql-action

change method used to check inclusion

Browse source
This commit is contained in:
Robert Brignull 2020-06-01 13:06:47 +01:00
parent 07e22b1f4a
commit 22501fd7c8
3 changed files with 7 additions and 5 deletions
Download patch file Download diff file Expand all files Collapse all files

5
lib/config-utils.js generated
View file

@ -60,8 +60,9 @@ class Config {
}
// Check for one of the builtin suites
if (queryUses.indexOf('/') === -1 && queryUses.indexOf('@') === -1) {
if (builtinSuites.includes(queryUses)) {
this.additionalSuites.push(queryUses);
const suite = builtinSuites.find((suite) => suite === queryUses);
if (suite) {
this.additionalSuites.push(suite);
return;
}
else {

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

File diff suppressed because one or more lines are too long

5
src/config-utils.ts
View file

@ -63,8 +63,9 @@ export class Config {
// Check for one of the builtin suites
if (queryUses.indexOf('/') === -1 && queryUses.indexOf('@') === -1) {
if (builtinSuites.includes(queryUses as any)) {
this.additionalSuites.push(queryUses as BuiltInSuite);
const suite = builtinSuites.find((suite) => suite === queryUses);
if (suite) {
this.additionalSuites.push(suite);
return;
} else {
throw new Error(getQueryUsesIncorrect(queryUses));