Repository navigation
ci: qualify next analysis checks for merge groups - #4150
Merged
Merged
Conversation
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. |
dpiet-oai
approved these changes
Oct 22, 2026
Contributor
Castiron custom codeEvaluated next: Trusted main policy: ✅ No new custom-code files detected. 48 mixed files remain; 0 existing customizations changed. Compared 48 existing customizations unchanged
8 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 39708538574 --repo openai/openai-python \
--name castiron-custom-code-39708538574-1 --dir /tmp/castiron-custom-code-39708538574-1
git apply --stat /tmp/castiron-custom-code-39708538574-1/custom-code.patch
cat /tmp/castiron-custom-code-39708538574-1/custom-code.patchOr reproduce it from an SDK checkout containing the vendored reporter: git fetch --no-tags origin b3eca25737d570ee277f77dcb6f10691f8656ef6 8c525da8b7c4dd7f31d75f63972d731e536464fb
python3 scripts/castiron/custom_code_report.py report \
--base b3eca25737d570ee277f77dcb6f10691f8656ef6 \
--head 8c525da8b7c4dd7f31d75f63972d731e536464fb --target-branch next --fetch --require-head-hash --public \
--out /tmp/castiron-custom-code-8c525da8b7c4
cat /tmp/castiron-custom-code-8c525da8b7c4/custom-code.patchThis is the current full custom patch for mixed files, not an attribution of only the handwritten lines changed by this PR. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Next pull requests and merge groups need CodeQL and breaking-change checks that cannot be satisfied by a main-target run sharing the same head. Run CodeQL for next PRs and merge groups, and run both existing breaking-change detectors against the merge group's exact base SHA with full history. Reject missing or unrelated baselines before dependency bootstrap.
Next PR/group runs publish
Analyze Python (next)anddetect-breaking-changes (next). Existing main check names remain; PR retargeting retriggers evaluation. Checkout credentials are not persisted. This changes check producers only; branch protection and merge-queue activation remain separate.Validation: actionlint passes both workflows. Focused tests cover seven event/target combinations using the actual expressions, release dispatch and edited triggers, real Git base validation, and the unchanged shell detector restoring base tests while retaining candidate SDK code. Actual hosted CodeQL and next merge-group qualification remain required before these contexts become required checks.