Skip to content

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

Merged
RobertCraigie merged 6 commits into
openai:nextfrom
Konnor-Young:main
Apr 22, 2025
Merged

fix(pydantic v1): more robust ModelField.annotation check#2163
RobertCraigie merged 6 commits into
openai:nextfrom
Konnor-Young:main

Conversation

@Konnor-Young

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

Copy link
Copy Markdown
Contributor
  • 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
@stainless-app stainless-app Bot mentioned this pull request Apr 22, 2025
@RobertCraigie

Copy link
Copy Markdown
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>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants