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
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: ✅ No new custom-code files detected. 40 mixed files remain; 0 existing customizations changed. Compared 40 existing customizations unchanged
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 38518027387 --repo openai/openai-python \
--name castiron-custom-code-38518027387-1 --dir /tmp/castiron-custom-code-38518027387-1
git apply --stat /tmp/castiron-custom-code-38518027387-1/custom-code.patch
cat /tmp/castiron-custom-code-38518027387-1/custom-code.patchOr reproduce it from an SDK checkout containing the vendored reporter: git fetch --no-tags origin 86cae50785fb7b7bf2a5d2383370b214b304acd2 ec49d7e7f4fd6003ad488d82f7505ff851b7f055
python3 scripts/castiron/custom_code_report.py report \
--base 86cae50785fb7b7bf2a5d2383370b214b304acd2 \
--head ec49d7e7f4fd6003ad488d82f7505ff851b7f055 --fetch --require-head-hash --public \
--out /tmp/castiron-custom-code-ec49d7e7f4fd
cat /tmp/castiron-custom-code-ec49d7e7f4fd/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: 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 the release-please version marker
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.
Raise the declared Griffe security floor
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 👍 / 👎.
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>
1269622 to
ec49d7e
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