| FazBrowse GitHub Viewer | Trending | | Home |
| Tools: [Download Repo ZIP] [Original HTTPS Page] |
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
…ion` Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
There was a problem hiding this comment.
Warning
The compatibility removals consistently follow the new enforced minimum and the relevant test matrices were updated.
Review effort: Balanced
Findings: None
Raises the minimum supported CodeQL CLI to 2.20.7 and removes compatibility paths for older releases.
Changes:
| File | Description |
|---|---|
| src/upload-sarif-action-post.ts | Simplifies debug artifact upload. |
| src/tools-features.ts | Removes obsolete safe-upload checks. |
| src/init-action.ts | Removes old Rust and Kotlin compatibility logic. |
| src/init-action-post-helper.ts | Removes the artifact-upload version parameter. |
| src/feature-flags.ts | Removes the obsolete C++ minimum version. |
| src/debug-artifacts.ts | Uploads artifacts without legacy version gating. |
| src/debug-artifacts.test.ts | Updates artifact-upload tests. |
| src/codeql.ts | Raises the minimum CLI version. |
| src/autobuild.ts | Removes unnecessary CLI dependency from C++ setup. |
| src/analyze.ts | Adapts C++ setup invocation. |
| src/analyze-action.ts | Removes the legacy proxy workaround. |
| src/analyze-action-post.ts | Avoids unnecessary CLI version lookup. |
| pr-checks/sync.ts | Updates the default tested versions. |
| pr-checks/checks/rust.yml | Tests Rust against the new minimum. |
| pr-checks/checks/multi-language-autodetect.yml | Drops the unsupported version. |
| package.json | Bumps the minor version. |
| package-lock.json | Synchronizes the package version. |
| CHANGELOG.md | Documents the breaking change. |
| .github/workflows/debug-artifacts-safe.yml | Updates debug-artifact coverage. |
| .github/workflows/debug-artifacts-failure-safe.yml | Updates failure-path coverage. |
| lib/entry-points.js | Generated output; contents excluded. |
| .github/workflows/__rust.yml | Generated workflow; contents excluded. |
| .github/workflows/__go-tracing-autobuilder.yml | Generated workflow; contents excluded. |
| .github/workflows/__multi-language-autodetect.yml | Generated workflow; contents excluded. |
| .github/workflows/__go-tracing-legacy-workflow.yml | Generated workflow; contents excluded. |
| .github/workflows/__go-tracing-custom-build-steps.yml | Generated workflow; contents excluded. |
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
Sorry, something went wrong.
There was a problem hiding this comment.
LGTM, changes look reasonable for the minimum version bump. Thanks for doing the clean-up!
Sorry, something went wrong.
…odeql-2-20-7 Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
| Back | FazBrowse Home | New Git URL |
Follows the deprecation in #3956.
All the tools features are still needed, since CodeQL 2.20.7 doesn't report any of them as supported. The README compatibility table and the next deprecation are handled in #4188.
Risk assessment
For internal use only. Please select the risk level of this change:
Which use cases does this change impact?
Workflow types:
Products:
Environments:
How did/will you validate this change?
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?
Are there any special considerations for merging or releasing this change?
Merge / deployment checklist