Treat processing error for multiple runs with the same category as configuration error by mbg · Pull Request #2987

Treat processing error for multiple runs with the same category as configuration error by mbg · Pull Request #2987 · github/codeql-action

Conversation

This will result in it being reported as a user error rather than a failure

Merge / deployment checklist

Confirm this change is backwards compatible with existing workflows.
Confirm the readme has been updated if necessary.
Confirm the changelog has been updated if necessary.

…nfiguration error

This will result in it being reported as a user error rather than a failure

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR updates error handling to treat "multiple runs with the same category" as a configuration error rather than a processing failure. This change ensures that such errors are reported as user errors instead of system failures.

Adds a new expected configuration error message to the error classification logic
Updates both TypeScript source and compiled JavaScript to maintain consistency

Reviewed Changes

Copilot reviewed 2 out of 3 changed files in this pull request and generated no comments.

File	Description
src/upload-lib.ts	Adds new error message to expectedConfigErrors array for proper classification
lib/upload-lib.js	Generated JavaScript equivalent of the TypeScript changes

Conversation

Merge / deployment checklist

Choose a reason for hiding this comment

Pull Request Overview

Reviewed Changes

Choose a reason for hiding this comment

3 participants