Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

Updated example keyword to json_schema_extra #233

Closed
wants to merge 3 commits into from

Conversation

jason-tang5
Copy link

Fixes #231

Changes

Received the following warning: "PydanticDeprecatedSince20: Using extra keyword arguments on Field is deprecated and will be removed. Use json_schema_extra instead. (Extra keys: 'example')." so we updated the example keyword to json_schema_extra.

One line description for the changelog

Updated example keyword to json_schema_extra

  • Tests pass
  • Appropriate changes to README are included in PR

Jason Tang and others added 3 commits February 10, 2024 12:11
Signed-off-by: Jason Tang <jasontang@Jasons-MacBook-Air-2.local>
Signed-off-by: Jason Tang <jasonliu.tang@mail.utoronto.ca>
@xSAVIKx
Copy link
Member

xSAVIKx commented May 26, 2024

Hey @jason-tang5, sorry it took me so long to review this.

I believe it should have been implemented using examples keyword instead and was covered by the other PR #235. I'm gonna close this one and sorry again for not getting back earlier.

@xSAVIKx xSAVIKx closed this May 26, 2024
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.

example should be examples in Field()?
2 participants