Repository navigation
chore(deps): bump CodeQL init and analyze to 4.38.2 - #4025
jbeckwith-oai merged 3 commits into
Conversation
Bumps [github/codeql-action/init](https://github.com/github/codeql-action) from 4.38.1 to 4.38.2. - [Release notes](https://github.com/github/codeql-action/releases) - [Changelog](https://github.com/github/codeql-action/blob/main/CHANGELOG.md) - [Commits](github/codeql-action@1c5b675...2892aa5) --- updated-dependencies: - dependency-name: github/codeql-action/init dependency-version: 4.38.2 dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <support@github.com>
OkTest Summary🟡 Running for Python SDK PR #4025. SDK merge (pending) · head ( |
Castiron custom codeEvaluated main: ✅ No new custom-code files detected. 46 mixed files remain; 0 existing customizations changed. Compared 46 existing customizations unchanged
6 more in the full report. A changed generated baseline means this report cannot reliably identify which handwritten lines changed. Inspect the custom-code diffDownload the exact patch produced by this run (requires repository access): gh run download 37945688704 --repo openai/openai-python \
--name castiron-custom-code-37945688704-1 --dir /tmp/castiron-custom-code-37945688704-1
git apply --stat /tmp/castiron-custom-code-37945688704-1/custom-code.patch
cat /tmp/castiron-custom-code-37945688704-1/custom-code.patchOr reproduce it from an SDK checkout containing the vendored reporter: git fetch --no-tags origin 8bfffd40158460d0f5e651590b95cfef5292d93e 3270213df647ab758914062c6c07d61da0903378
python3 scripts/castiron/custom_code_report.py report \
--base 8bfffd40158460d0f5e651590b95cfef5292d93e \
--head 3270213df647ab758914062c6c07d61da0903378 --fetch --require-head-hash --public \
--out /tmp/castiron-custom-code-3270213df647
cat /tmp/castiron-custom-code-3270213df647/custom-code.patchThis is the current full custom patch for mixed files, not an attribution of only the handwritten lines changed by this PR. |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: ab2a663200
ℹ️ About Codex in GitHub
Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".
|
|
||
| - name: Initialize CodeQL | ||
| uses: github/codeql-action/init@1c5b675653bb5c22dbe9b12b556ec555138e09fd # v4.38.1 | ||
| uses: github/codeql-action/init@2892aa5e19bbd11bc0cff5427e3b750a04d9e3c2 # v4.38.2 |
There was a problem hiding this comment.
Keep CodeQL init and analyze on the same release
This upgrades only init; the subsequent analyze step remains pinned to v4.38.1. CodeQL explicitly does not support mixing action versions in one workflow, and its version validation can error when analyze loads initialization state written by a different init version, so this can break every CodeQL run after the update. Update github/codeql-action/analyze to the v4.38.2 SHA as part of this bump. CodeQL Action changelog
Useful? React with 👍 / 👎.
|
Fixed the CodeQL failure by updating init and analyze together to the same reviewed full SHA for v4.38.2. The original job failed because its configuration was created by 4.38.2 but read by 4.38.1. This PR now supersedes #4024. Permissions, scanned languages, and SDK code remain unchanged. Upstream release/tag and changes reviewed; two consecutive independent clean review rounds completed. Watching fresh CI, including the prior unrelated Castiron temporary-directory cleanup failure. |
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
jbeckwith-oai
left a comment
There was a problem hiding this comment.
Approved after review of 3270213 and green exact-head CI. Both CodeQL steps now use the same verified immutable v4.38.2 SHA, fixing the demonstrated mixed-version configuration failure. Upstream changelog and changes reviewed. Workflow permissions, language selection, release boundaries, SDK code, and published dependencies remain unchanged. All CI checks pass, including CodeQL, Python 3.10/3.14, HTTPX2, build, lint and policy/custom-code checks. Two consecutive independent clean review rounds; merging current main preserved the reviewed patch ID. Supersedes closed #4024. Ready for normal protected merge.
Automated Release PR --- ## [3.28.0](openai/openai-python@v3.27.0...v3.28.0) (2026-10-09) ### Features * **api:** add agent environment suspension and expiration ([openai#4046](openai#4046)) ([6aa25a9](openai@6aa25a9)) ### Chores * **deps:** bump astral-sh/setup-uv from 10.1.0 to 10.2.0 ([openai#4026](openai#4026)) ([8bfffd4](openai@8bfffd4)) * **deps:** bump CodeQL init and analyze to 4.38.2 ([openai#4025](openai#4025)) ([4f5e618](openai@4f5e618)) * **deps:** update PyJWT security constraint to 2.14.0 ([openai#3991](openai#3991)) ([aaed2a7](openai@aaed2a7)) --- This PR was generated with [Release Please](https://github.com/googleapis/release-please). See [documentation](https://github.com/googleapis/release-please#release-please). Co-authored-by: openai-sdks[bot] <284451331+openai-sdks[bot]@users.noreply.github.com>
Current scope
Update both CodeQL init and analyze to the same reviewed v4.38.2 SHA. Updating only one step caused a configuration-version mismatch. This supersedes the separate analyze PR #4024. Workflow permissions and SDK behavior remain unchanged.
Bumps github/codeql-action/init from 4.38.1 to 4.38.2.
Release notes
Sourced from github/codeql-action/init's releases.
Changelog
Sourced from github/codeql-action/init's changelog.
... (truncated)
Commits
2892aa5Merge pull request #4168 from github/update-v4.38.2-a6ef2c96f8ad03a3Trigger workflows98af865Update changelog for v4.38.2a6ef2c9Merge pull request #4156 from github/mario-campos/fix-validate-cmd1ef28a1Merge pull request #4166 from github/dependabot/github_actions/dot-github/wor...26cb08bMerge pull request #4163 from github/mbg/fix-getCommitOid-stubsf035ce3Merge pull request #4165 from github/dependabot/npm_and_yarn/npm-minor-8eaed9...5e4e255Rebuildb13f5f4Bump ruby/setup-rubyc87fe57RebuildDependabot 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)