Skip to content

fix(pydantic v1): more robust ModelField.annotation check #2163

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Merged
merged 6 commits into from
Apr 22, 2025

Conversation

Konnor-Young
Copy link
Contributor

@Konnor-Young Konnor-Young commented Mar 5, 2025

  • I understand that this repository is auto-generated and my pull request may not be merged

Changes being requested

Made a change to check the field_info for 'annotation' prior to accessing. In an attempt to prevent the "'ModelField' object has no attribute 'annotation'" error.

Additional context & links

closes issue #2155

The issue mentions running into this error on library version 1.52.0
I was running into this error up to version 1.65.3

pydantic version 1.9.0
python version 3.9

@Konnor-Young Konnor-Young requested a review from a team as a code owner March 5, 2025 17:56
@RobertCraigie RobertCraigie changed the title Fix: 'ModelField' object has no attribute 'annotation' fix(pydantic v1): more robust ModelField.annotation check Apr 22, 2025
@RobertCraigie RobertCraigie changed the base branch from main to next April 22, 2025 21:18
@RobertCraigie RobertCraigie merged commit 7351b12 into openai:next Apr 22, 2025
1 of 3 checks passed
@stainless-app stainless-app bot mentioned this pull request Apr 22, 2025
@RobertCraigie
Copy link
Collaborator

Thanks!

stainless-app bot pushed a commit that referenced this pull request Apr 23, 2025
---------

Co-authored-by: Konnor Young <konnoryoung@ip-192-168-1-39.us-west-1.compute.internal>
Co-authored-by: Robert Craigie <robert@craigie.dev>
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants