Skip to content

Conversation

@nickrolfe
Copy link
Contributor

@nickrolfe nickrolfe commented Feb 3, 2026

No description provided.

@nickrolfe nickrolfe force-pushed the nickrolfe/rm-old-version branch from 9719968 to 8864622 Compare February 3, 2026 10:21
@nickrolfe nickrolfe marked this pull request as ready for review February 3, 2026 10:52
@nickrolfe nickrolfe requested a review from a team as a code owner February 3, 2026 10:52
Copilot AI review requested due to automatic review settings February 3, 2026 10:52
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.

Pull request overview

This PR removes support for CodeQL CLI versions older than 2.21.4, updating the minimum supported version from 2.20.7.

Changes:

  • Updated the minimum supported CLI version to 2.21.4
  • Removed v2.20.7 from the list of supported CLI versions
  • Added changelog entry documenting the change

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.

File Description
extensions/ql-vscode/supported_cli_versions.json Removed v2.20.7 from the list of supported CLI versions
extensions/ql-vscode/src/codeql-cli/cli.ts Updated OLDEST_SUPPORTED_CLI_VERSION constant from 2.20.7 to 2.21.4
extensions/ql-vscode/CHANGELOG.md Added changelog entry for version support change, plus minor unrelated whitespace corrections
Comments suppressed due to low confidence (1)

extensions/ql-vscode/CHANGELOG.md:200

  • This spacing correction (removing an extra space after the period) appears to be unrelated to the PR's purpose of removing support for CodeQL CLI versions older than 2.21.4. While this is a minor formatting improvement, consider whether it should be included in this PR or addressed separately to keep the PR focused on a single change.
- No longer highlight trailing commas for jump to definition. [#2615](https://github.com/github/vscode-codeql/pull/2615)

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@nickrolfe nickrolfe force-pushed the nickrolfe/rm-old-version branch from 8864622 to 50e97aa Compare February 3, 2026 10:58
@nickrolfe nickrolfe merged commit 15aae38 into main Feb 3, 2026
32 checks passed
@nickrolfe nickrolfe deleted the nickrolfe/rm-old-version branch February 3, 2026 11:29
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.

5 participants