-
Notifications
You must be signed in to change notification settings - Fork 409
Add setup-codeql action
#3204
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add setup-codeql action
#3204
Conversation
There was a problem hiding this comment.
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 introduces a new experimental setup-codeql action that installs the CodeQL CLI without initializing a database, providing a more focused tool setup capability separate from the existing init action.
Key changes:
- Adds a standalone
setup-codeqlaction for CodeQL CLI installation only - Creates new action entry point with status reporting integration
- Updates test workflows to use the new action instead of
init
Reviewed Changes
Copilot reviewed 7 out of 8 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
src/status-report.ts |
Adds SetupCodeQL action name enum for telemetry tracking |
src/setup-codeql-action.ts |
New action implementation that installs CodeQL CLI and sets outputs |
setup-codeql/action.yml |
Action definition with inputs/outputs for the new setup action |
pr-checks/checks/bundle-from-toolcache.yml |
Updates test to use setup-codeql instead of init |
README.md |
Documents the new experimental action |
CHANGELOG.md |
Records the addition of the new action |
.github/workflows/__bundle-from-toolcache.yml |
Generated workflow file reflecting test changes |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice! If we aren't yet supporting running init after setup-codeql, should it be a configuration error to do so?
This PR takes a stab at adding a
setup-codeqlaction, which just installs the CodeQL CLI. Unlikeinit, it does not initialise a database.This doesn't change the
initaction in any way yet. I.e. the goal of this PR is not to support runningsetup-codeqlfollowed byinit, but justsetup-codeqlon its own with no subsequentcodeql-actionsteps.Risk assessment
For internal use only. Please select the risk level of this change:
Which use cases does this change impact?
Currently, none.
How did/will you validate this change?
.test.tsfiles).pr-checks).If something goes wrong after this change is released, what are the mitigation and rollback strategies?
How will you know if something goes wrong after this change is released?
Merge / deployment checklist