Skip to content

Allow the CodeQL CLI path to be set from an environment variable#3118

Merged
charisk merged 3 commits intomainfrom
charisk/stop-downloading-cli
Dec 14, 2023
Merged

Allow the CodeQL CLI path to be set from an environment variable#3118
charisk merged 3 commits intomainfrom
charisk/stop-downloading-cli

Conversation

@charisk
Copy link
Copy Markdown
Contributor

@charisk charisk commented Dec 7, 2023

This is to avoid downloading the CodeQL CLI in each test suite

Checklist

N/A:

  • CHANGELOG.md has been updated to incorporate all user visible changes made by this pull request.
  • Issues have been created for any UI or other user-facing changes made by this pull request.
  • [Maintainers only] If this pull request makes user-facing changes that require documentation changes, open a corresponding docs pull request in the github/codeql repo and add the ready-for-doc-review label there.

@charisk charisk requested a review from a team as a code owner December 7, 2023 16:40
Copy link
Copy Markdown
Member

@koesie10 koesie10 left a comment

Choose a reason for hiding this comment

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

Thanks, this looks like a good solution to me

@charisk charisk force-pushed the charisk/stop-downloading-cli branch from 0774912 to 188bae1 Compare December 13, 2023 16:25
@charisk charisk force-pushed the charisk/stop-downloading-cli branch from 2c6fde4 to b122caa Compare December 13, 2023 16:25
@charisk charisk enabled auto-merge (squash) December 14, 2023 08:50
@charisk charisk merged commit 598f2eb into main Dec 14, 2023
@charisk charisk deleted the charisk/stop-downloading-cli branch December 14, 2023 09:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants