| FazBrowse GitHub Viewer | Trending | | Home |
| Tools: [Download Repo ZIP] [Original HTTPS Page] |
There was a problem hiding this comment.
Updates overlay analysis resource checking so that the minimum-memory gating is skipped for CodeQL CLI versions where overlay no longer increases peak RAM usage (2.24.3+).
Changes:
Copilot reviewed 3 out of 3 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
| src/config-utils.ts | Skips overlay minimum-memory check for CodeQL >= 2.24.3 and refactors disk/memory resource checks. |
| src/config-utils.test.ts | Adds test cases for the new version-gated memory-check behavior on default branch and PR runs. |
| lib/init-action.js | Generated JS update corresponding to the TypeScript changes (not reviewed). |
Sorry, something went wrong.
There was a problem hiding this comment.
LGTM!
Sorry, something went wrong.
The base branch was changed.
|
Rebased on main |
Sorry, something went wrong.
Skip overlay memory check for CodeQL 2.24.3 and later
| Back | FazBrowse Home | New Git URL |
RAM usage with overlay analysis enabled should generally be no higher than it is without overlay analysis for these versions.
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?
These changes will be validated as part of the next release process.
If something goes wrong after this change is released, what are the mitigation and rollback strategies?
Disable via broader overlay feature flags, or CodeQL version feature flags.
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