| FazBrowse GitHub Viewer | Trending | | Home |
| Tools: [Download Repo ZIP] [Original HTTPS Page] |
Sorry, something went wrong.
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
A maintainer should confirm rollout readiness before removing the feature-flag kill switch.
1 open finding What changed in this PRGraduates overlay-aware default CodeQL version selection for eligible Code Scanning pull requests.
Changes:
| File | Description |
|---|---|
| src/setup-codeql.ts | Graduates selection and adds cache-lookup guards. |
| src/setup-codeql.test.ts | Covers supported environments and bypass conditions. |
| src/feature-flags.ts | Removes selection and dry-run flags. |
🧠 Review effort: Max
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
Sorry, something went wrong.
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
…overlay-version-match Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
There was a problem hiding this comment.
The changes make sense and thanks for the explanation of the drive-by improvement in the PR description. Just one minor comment about the rather large condition in getEnabledVersionsWithOverlayBaseDatabases.
Sorry, something went wrong.
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
There was a problem hiding this comment.
Thanks for simplifying / documenting that!
Sorry, something went wrong.
…overlay-version-match Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
| Back | FazBrowse Home | New Git URL |
Graduate overlay_analysis_match_codeql_version and remove the overlay_analysis_match_codeql_version_dry_run dry run, along with the telemetry diagnostic added for it. Also, as a small optimisation, skip listing the Actions cache when only one default CodeQL version is enabled, since there is only one CodeQL version to choose in this case. This includes GHES, since there are no feature flags and therefore only one CodeQL version will ever be enabled there.
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