Skip to content

Commit

Permalink
Moves calls to pack download to the init action
Browse files Browse the repository at this point in the history
This ensures all steps to gather queries happens in the init action.
This is where checking out queries in other repos happens as well.
  • Loading branch information
aeisenberg committed Aug 26, 2022
1 parent a59fbe2 commit 1e0fc04
Show file tree
Hide file tree
Showing 9 changed files with 156 additions and 28 deletions.
9 changes: 0 additions & 9 deletions lib/analyze.js

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

2 changes: 1 addition & 1 deletion lib/analyze.js.map

Large diffs are not rendered by default.

23 changes: 23 additions & 0 deletions lib/config-utils.js

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

2 changes: 1 addition & 1 deletion lib/config-utils.js.map

Large diffs are not rendered by default.

48 changes: 48 additions & 0 deletions lib/config-utils.test.js

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

2 changes: 1 addition & 1 deletion lib/config-utils.test.js.map

Large diffs are not rendered by default.

15 changes: 0 additions & 15 deletions src/analyze.ts
Original file line number Diff line number Diff line change
Expand Up @@ -280,21 +280,6 @@ export async function runQueries(
logger.endGroup();
logger.info(analysisSummary);
} else {
if (hasPackWithCustomQueries) {
logger.info("Performing analysis with custom CodeQL Packs.");
logger.startGroup(`Downloading custom packs for ${language}`);

const results = await codeql.packDownload(packsWithVersion);

logger.info(
`Downloaded packs: ${results.packs
.map((r) => `${r.name}@${r.version || "latest"}`)
.join(", ")}`
);

logger.endGroup();
}

logger.startGroup(`Running queries for ${language}`);
const querySuitePaths: string[] = [];
if (queries["builtin"].length > 0) {
Expand Down
Loading

0 comments on commit 1e0fc04

Please sign in to comment.