Repository navigation
chore(deps-dev): bump griffe from 1.14.0 to 2.3.0 - #4069
dependabot[bot] wants to merge 1 commit into
Conversation
dependabot
Bot
added
dependencies
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. |
Castiron custom codeEvaluated main: Trusted main policy: ✅ No new custom-code files detected. 41 mixed files remain; 0 existing customizations changed. Compared 41 existing customizations unchanged
1 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 39085368766 --repo openai/openai-python \
--name castiron-custom-code-39085368766-1 --dir /tmp/castiron-custom-code-39085368766-1
git apply --stat /tmp/castiron-custom-code-39085368766-1/custom-code.patch
cat /tmp/castiron-custom-code-39085368766-1/custom-code.patchOr reproduce it from an SDK checkout containing the vendored reporter: git fetch --no-tags origin 547ecb1e4fc233f19030736c29fee4374dfbaca1 c7cc4f5559b2658c2a19f279c07c72ab1f9fa1a1
python3 scripts/castiron/custom_code_report.py report \
--base 547ecb1e4fc233f19030736c29fee4374dfbaca1 \
--head c7cc4f5559b2658c2a19f279c07c72ab1f9fa1a1 --fetch --require-head-hash --public \
--out /tmp/castiron-custom-code-c7cc4f5559b2
cat /tmp/castiron-custom-code-c7cc4f5559b2/custom-code.patchThis is the current full custom patch for mixed files, not an attribution of only the handwritten lines changed by this PR. |
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: de69acdec7
ℹ️ 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".
| [[package]] | ||
| name = "openai" | ||
| version = "3.29.0" # x-release-please-version | ||
| version = "3.29.0" |
There was a problem hiding this comment.
Restore # x-release-please-version on this version line. The dependency-locks job in .github/workflows/ci.yml:63-67 explicitly asserts that uv.lock contains the exact marked line, so every CI run for this commit fails before lint, build, and tests; the same assertion fails locally against the committed lock.
Useful? React with 👍 / 👎.
de69acd to
1269622
Compare
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 1269622246
ℹ️ 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".
| [[package]] | ||
| name = "griffe" | ||
| version = "1.14.0" | ||
| version = "2.3.0" |
There was a problem hiding this comment.
When the dependency-locks job runs the base revision's scripts/check-dependency-security.py before lock validation, this upgrade is compared with the unchanged griffe>=1 declaration in pyproject.toml; running that checker against the parent and this tree exits with Raise the contextual protected security minimum to the patched release for griffe. Update the declared dev-dependency minimum to cover 2.3.0, otherwise CI still fails even after the separately reported release-please marker is restored.
AGENTS.md reference: AGENTS.md:L31-L34
Useful? React with 👍 / 👎.
6b9b607 to
6611468
Compare
Bumps [griffe](https://github.com/mkdocstrings/griffe) from 1.14.0 to 2.3.0. - [Release notes](https://github.com/mkdocstrings/griffe/releases) - [Changelog](https://github.com/mkdocstrings/griffe/blob/main/CHANGELOG.md) - [Commits](mkdocstrings/griffe@1.14.0...2.3.0) --- updated-dependencies: - dependency-name: griffe dependency-version: 2.3.0 dependency-type: direct:development update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <support@github.com>
6611468 to
c7cc4f5
Compare
Bumps griffe from 1.14.0 to 2.3.0.
Release notes
Sourced from griffe's releases.
... (truncated)
Changelog
Sourced from griffe's changelog.
... (truncated)
Commits
ecd5349chore: Prepare release 2.3.009a2a13feat: Add option to give precedence to docstrings in stubs vs. sourcee095a7cfix: Preserve nested tuple parenthesesefa8ac1Merge branch 'main' of github.com:mkdocstrings/griffe092ac31perf: Reuse computed parent inis_imported76e1082chore: Add docs for package parameter8f05a4aperf: Only build debug messages at debug level06cd02eperf: Avoid creating new sets when handling labelse21851eperf: Maintain map of AST node type to visitor functionb968de0perf: Avoid cyclic refs in ASTs to allow garbage-collecting them