-
Notifications
You must be signed in to change notification settings - Fork 356
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
Remove bundled copy of actions
extractor
#2787
base: main
Are you sure you want to change the base?
Conversation
To be safe, I checked our telemetry and there have been zero runs in the last 14 days that analyzed |
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.
Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.
🤦♂️ The |
I will pick this PR up again. |
In #2582, I temporarily added an embedded version of the
actions
extractor into the CodeQL Action itself, to work around the extractor being omitted from the CLI bundle. That's been fully corrected as of CLI 2.20.4, so we should no longer need the embedded copy.I'm ignoring the possibility of Advanced Setup users having pinned
tools:
to a specific version, since the Actions analysis is only in preview anyway.Merge / deployment checklist