Skip to content

Update actions query suites#19002

Merged
aeisenberg merged 2 commits intomainfrom
aeisenberg/actions-security-and-quality
Mar 12, 2025
Merged

Update actions query suites#19002
aeisenberg merged 2 commits intomainfrom
aeisenberg/actions-security-and-quality

Conversation

@aeisenberg
Copy link
Copy Markdown
Contributor

Previously, the actions-security-and-quality.qls and actions-security-experimental.qlssuites were not using the proper selectors. Now they are the same as in other languages.

It was not using the proper selectors. It is now the same as
other quality suites in other languages.
Copilot AI review requested due to automatic review settings March 12, 2025 18:48
@aeisenberg aeisenberg requested a review from a team as a code owner March 12, 2025 18:48
Copy link
Copy Markdown
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.

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

Tip: If you use Visual Studio Code, you can request a review from Copilot before you push from the "Source Control" tab. Learn more

@github-actions github-actions Bot added the Actions Analysis of GitHub Actions label Mar 12, 2025
@adityasharad
Copy link
Copy Markdown
Collaborator

Change looks okay, but checking why we need it - at one point we deliberately kept these suites identical, now we need this change because we are adding queries to security-and-quality or security-experimental that will not be in security-extended?

@aeisenberg
Copy link
Copy Markdown
Contributor Author

Yes. I added a few queries that I want to be in security-experimental and one that I want to be in security-and-quality.

@aeisenberg aeisenberg merged commit e05b172 into main Mar 12, 2025
@aeisenberg aeisenberg deleted the aeisenberg/actions-security-and-quality branch March 12, 2025 20:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Actions Analysis of GitHub Actions

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants