Skip to content

Commit 64ec68a

Browse files
committed
Include diagnostics in bundle
1 parent 6cff5d3 commit 64ec68a

13 files changed

+81
-27
lines changed

lib/analyze-action-post.js

Lines changed: 6 additions & 2 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

lib/analyze-action.js

Lines changed: 11 additions & 4 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

lib/autobuild-action.js

Lines changed: 6 additions & 2 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

lib/init-action-post.js

Lines changed: 10 additions & 4 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

lib/init-action.js

Lines changed: 9 additions & 5 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

lib/resolve-environment-action.js

Lines changed: 6 additions & 2 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

lib/setup-codeql-action.js

Lines changed: 6 additions & 2 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

lib/upload-lib.js

Lines changed: 6 additions & 2 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

lib/upload-sarif-action.js

Lines changed: 6 additions & 2 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

src/codeql.ts

Lines changed: 9 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -160,6 +160,7 @@ export interface CodeQL {
160160
databasePath: string,
161161
outputFilePath: string,
162162
dbName: string,
163+
includeDiagnostics: boolean,
163164
alsoIncludeRelativePaths: string[],
164165
): Promise<void>;
165166
/**
@@ -912,15 +913,22 @@ async function getCodeQLForCmd(
912913
databasePath: string,
913914
outputFilePath: string,
914915
databaseName: string,
916+
includeDiagnostics: boolean,
915917
alsoIncludeRelativePaths: string[],
916918
): Promise<void> {
919+
const includeDiagnosticsArgs = includeDiagnostics
920+
? ["--include-diagnostics"]
921+
: [];
917922
const args = [
918923
"database",
919924
"bundle",
920925
databasePath,
921926
`--output=${outputFilePath}`,
922927
`--name=${databaseName}`,
923-
...getExtraOptionsFromEnv(["database", "bundle"]),
928+
...includeDiagnosticsArgs,
929+
...getExtraOptionsFromEnv(["database", "bundle"], {
930+
ignoringOptions: includeDiagnosticsArgs,
931+
}),
924932
];
925933
if (
926934
await this.supportsFeature(ToolsFeature.BundleSupportsIncludeOption)

0 commit comments

Comments
 (0)