Skip to content

Conversation

@aeisenberg
Copy link
Contributor

Add all of the codeql-action actions.

Add all of the codeql-action actions.
@aeisenberg aeisenberg requested a review from a team as a code owner February 26, 2025 19:19
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 updates the immutable_actions.yml file to include all of the GitHub CodeQL Action actions.

  • Added several CodeQL Action steps to the configuration
  • Updated the list of immutable actions to reflect new functionality

Reviewed Changes

File Description
actions/ql/lib/ext/config/immutable_actions.yml Added new CodeQL Action steps to the immutable actions configuration

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

@github-actions github-actions bot added the Actions Analysis of GitHub Actions label Feb 26, 2025
@aeisenberg
Copy link
Contributor Author

aeisenberg commented Feb 26, 2025

I should add a change note.

Actually, I don't think so. This query is already only meant for internal use. When immutable actions are released publicly, any changes to the data extension file should come with a change note, but until then it's not necessary.

@aeisenberg aeisenberg added the no-change-note-required This PR does not need a change note label Feb 26, 2025
Copy link

@dbartol dbartol left a comment

Choose a reason for hiding this comment

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

Since we've just decided to move the default list of immutable actions into an internal-only model pack until Immutable Actions is available to customers, I'd prefer not to merge this PR just yet. Instead, I'll create that model pack, but include the CodeQL actions in that list.

@aeisenberg
Copy link
Contributor Author

Sounds good.

@aeisenberg aeisenberg closed this Feb 27, 2025
@dbartol
Copy link

dbartol commented Feb 27, 2025

See #18887

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Actions Analysis of GitHub Actions no-change-note-required This PR does not need a change note

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants