Skip to content

Conversation

@rvermeulen
Copy link
Contributor

Ensure the CCR suite is recorded in the query list for automation purposes.

Copilot AI review requested due to automatic review settings February 6, 2025 22:11
@rvermeulen rvermeulen requested a review from RasmusWL as a code owner February 6, 2025 22:11
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

PR Overview

This pull request adds the "ccr" suite to the list of query packs and updates logic to handle potentially empty query results for the CCR suite. The primary changes include stripping query results earlier to detect empty suites and skipping them if they are empty.

  • Adds "ccr" to the packs array.
  • Strips query results before splitting to manage empty suite logic.
  • Skips empty CCR suite instead of raising an error.

Changes

File Description
misc/scripts/generate-code-scanning-query-list.py Adds CCR to the query packs and gracefully handles an empty CCR suite.

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

Tip: Copilot only keeps its highest confidence comments to reduce noise and keep you focused. Learn more

Copy link
Contributor

@aeisenberg aeisenberg left a comment

Choose a reason for hiding this comment

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

LGTM

@rvermeulen rvermeulen merged commit 9971398 into github:main Feb 10, 2025
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants