build(deps): bump github/codeql-action from 4 to 4.37.3 - #156
build(deps): bump github/codeql-action from 4 to 4.37.3#156dependabot[bot] wants to merge 1 commit into
Conversation
Bumps [github/codeql-action](https://github.com/github/codeql-action) from 4 to 4.37.3. - [Release notes](https://github.com/github/codeql-action/releases) - [Changelog](https://github.com/github/codeql-action/blob/main/CHANGELOG.md) - [Commits](github/codeql-action@v4...v4.37.3) --- updated-dependencies: - dependency-name: github/codeql-action dependency-version: 4.37.3 dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <support@github.com>
|
Codex review: needs maintainer review before merge. Reviewed August 1, 2026, 7:48 PM ET / 23:48 UTC. ClawSweeper reviewWhat this changesPins the CodeQL workflow’s initialization and analysis actions from the floating v4 tag to the released v4.37.3 tag. Merge readinessThis PR remains necessary: current main and the latest release still use the floating CodeQL v4 action tag, while this branch pins both CodeQL steps to v4.37.3. The focused update is correct and PR-head scans succeeded, but the declared workflow code owner should make the normal acceptance decision because it changes the runtime used for future security scans. Priority: P3 Review scores
Verification
How this fits togetherThe CodeQL GitHub Actions workflow runs on pull requests, pushes, scheduled scans, and manual dispatches to analyze the repository’s Go code and publish security results. This PR changes the version of the external CodeQL action that performs its initialization and analysis steps. flowchart LR
A[GitHub events] --> B[CodeQL workflow]
B --> C[CodeQL initialization]
C --> D[Go code analysis]
D --> E[Security scan results]
F[Action version pin] --> C
F --> D
Decision needed
Why: The patch is mechanically sound and has successful PR-head scan evidence, but it intentionally changes the security-scanning runtime and freezes future v4 patch movement until another update lands. Before merge
Agent review detailsSecurityNone. Review metrics
Merge-risk optionsMaintainer options:
Technical reviewBest possible solution: Approve the paired v4.37.3 pin after the workflow owner accepts the normal rollback responsibility, then continue consuming future CodeQL action updates through focused dependency PRs. Do we have a high-confidence way to reproduce the issue? Not applicable—the PR is a dependency-maintenance update rather than a bug report; the successful PR-head CodeQL run directly exercises the changed workflow path. Is this the best way to solve the issue? Yes—the two references are the complete CodeQL action surface in this workflow, and updating them together to one released version is the narrowest maintainable change. AGENTS.md: found and applied where relevant. Codex review notes: model internal, reasoning high; reviewed against 8141387a4f95. LabelsLabel justifications:
EvidenceWhat I checked:
Likely related people:
Rating scale
Overall follows the weaker of proof and patch quality. Workflow
HistoryReview history (5 earlier review cycles)
|
|
This PR is already one patch behind: CodeQL Action v4.37.4 was released on 2026-07-30, while this branch targets v4.37.3. #158 updates both CodeQL steps to the current v4.37.4 release and includes the rest of the repository's dependency refresh, with full local gates and live CLI proof. Recommendation: close this PR as superseded by #158, then land #158 after its CI is green. |
|
Looks like github/codeql-action is up-to-date now, so this is no longer needed. |
|
#158 landed as |
Bumps github/codeql-action from 4 to 4.37.3.
Release notes
Sourced from github/codeql-action's releases.
... (truncated)
Changelog
Sourced from github/codeql-action's changelog.
Commits
e4fba86Merge pull request #4031 from github/update-v4.37.3-72f6a9da0fb50ab5Update changelog for v4.37.372f6a9dMerge pull request #4030 from github/mbg/fix/no-proxy3b5ee58Use defaultrequestoptions instead ofundefinedbfb6be4Merge pull request #4028 from github/mergeback/v4.37.2-to-main-e0647621526ab84Rebuildd6217b9Update changelog and version after v4.37.2e064762Merge pull request #4027 from github/update-v4.37.2-385bcdc5ae0faed8Add a couple of change notes73aad0eUpdate changelog for v4.37.2Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting
@dependabot rebase.Dependabot commands and options
You can trigger Dependabot actions by commenting on this PR:
@dependabot rebasewill rebase this PR@dependabot recreatewill recreate this PR, overwriting any edits that have been made to it@dependabot show <dependency name> ignore conditionswill show all of the ignore conditions of the specified dependency@dependabot ignore this major versionwill close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)@dependabot ignore this minor versionwill close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)@dependabot ignore this dependencywill close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)