Remove --sarif-include-diagnostics from diagnotics export

This commit is contained in:
Angela P Wen
2023-03-13 16:18:03 -07:00
parent 1e4efb175e
commit ec89543aba
3 changed files with 7 additions and 26 deletions
+3 -17
View File
@@ -1013,7 +1013,9 @@ export async function getCodeQLForCmd(
"--db-cluster", // Database is always a cluster for CodeQL versions that support diagnostics.
"--format=sarif-latest",
`--output=${sarifFile}`,
await getSarifIncludeDiagnosticsArgument(this, features),
(await features.getValue(Feature.ExportCodeScanningConfigEnabled, this))
? "--sarif-include-diagnostics"
: "",
...getExtraOptionsFromEnv(["diagnostics", "export"]),
];
@@ -1034,7 +1036,6 @@ export async function getCodeQLForCmd(
"export",
"--format=sarif-latest",
`--output=${sarifFile}`,
await getSarifIncludeDiagnosticsArgument(this, features),
...(await getCodeScanningConfigExportArguments(config, this, features)),
...getExtraOptionsFromEnv(["diagnostics", "export"]),
];
@@ -1273,18 +1274,3 @@ async function getCodeScanningConfigExportArguments(
}
return [];
}
async function getSarifIncludeDiagnosticsArgument(
codeql: CodeQL,
features: FeatureEnablement
): Promise<string> {
if (
(await features.getValue(
Feature.ExportCodeScanningConfigEnabled,
codeql
)) === true
) {
return "--sarif-include-diagnostics";
}
return "";
}