You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This PR takes a stab at adding a setup-codeql action, which just installs the CodeQL CLI. Unlike init, it does not initialise a database.
This doesn't change the init action in any way yet. I.e. the goal of this PR is not to support running setup-codeql followed by init, but just setup-codeql on its own with no subsequent codeql-action steps.
Risk assessment
For internal use only. Please select the risk level of this change:
Low risk: Changes are fully under feature flags, or have been fully tested and validated in pre-production environments and are highly observable, or are documentation or test only.
Which use cases does this change impact?
Currently, none.
How did/will you validate this change?
Unit tests - I am depending on unit test coverage (i.e. tests in .test.ts files).
End-to-end tests - I am depending on PR checks (i.e. tests in pr-checks).
If something goes wrong after this change is released, what are the mitigation and rollback strategies?
Rollback - Change can only be disabled by rolling back the release or releasing a new version with a fix.
How will you know if something goes wrong after this change is released?
Telemetry - I rely on existing telemetry or have made changes to the telemetry.
Dashboards - I will watch relevant dashboards for issues after the release. Consider whether this requires this change to be released at a particular time rather than as part of a regular release.
Merge / deployment checklist
Confirm this change is backwards compatible with existing workflows.
Consider adding a changelog entry for this change.
Confirm the readme and docs have been updated if necessary.
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-codeql action 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
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
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