Skip to content

Commit e45c9a6

Browse files
committed
Stop using exit code 32 for new category
This clashes with the logic for the `NoJavaScriptTypeScriptCodeFound` case. Once that situation is deprecated, we can add the exit code back to `DetectedCodeButCouldNotProcess` configuration.
1 parent 9bc1f4d commit e45c9a6

3 files changed

Lines changed: 5 additions & 5 deletions

File tree

lib/cli-errors.js

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

lib/cli-errors.js.map

Lines changed: 1 addition & 1 deletion
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

src/cli-errors.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -130,7 +130,6 @@ export const cliErrorsConfig: Record<
130130
// Usually when a manual build script has failed, or if an autodetected language
131131
// was unintended to have CodeQL analysis run on it.
132132
[CliConfigErrorCategory.DetectedCodeButCouldNotProcess]: {
133-
exitCode: 32,
134133
cliErrorMessageSnippets: [
135134
"CodeQL detected code written in",
136135
"but could not process any of it",
@@ -158,7 +157,8 @@ export const cliErrorsConfig: Record<
158157
* `codeql database finalize`. To ensure users get a good error message, we detect this manually
159158
* here, and upon detection override the error message.
160159
*
161-
* This can be removed once support for CodeQL 2.11.6 is removed.
160+
* This can be removed once support for CodeQL 2.11.6 is removed, and once removed, exit code 32
161+
* can be applied to the DetectedCodeButCouldNotProcess category.
162162
*/
163163
[CliConfigErrorCategory.NoJavaScriptTypeScriptCodeFound]: {
164164
exitCode: 32,

0 commit comments

Comments
 (0)