-
Notifications
You must be signed in to change notification settings - Fork 97
441 update pydantic section of docs #442
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
Conversation
Hi @jbkroner, are you ready with the PR or do you want to amend it? Just asking as the "Draft" label here is still in place and I didn't want to merge something unfinished. 😉 Thanks, looks good! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@jbkroner Just a small, minor suggestion. Apart from that, it looks good! 👍
@tomschr I applied your feedback - keeping this in a draft state until the changes referenced in this update are included in an official |
9f365e8
to
2a2d08a
Compare
2a2d08a
to
bdca464
Compare
@jbkroner - |
Co-authored-by: Tom Schraitle <tomschr@users.noreply.github.com>
Co-authored-by: Tom Schraitle <tomschr@users.noreply.github.com>
This type was included in pydantic-extra-types 2.10 . There may have been some updates since it was included that may have slightly changed behavior, I'm checking that now. |
Great, thanks @jbkroner! Merged it. 👍 |
Closes #441
Small docs update, includes updated guidance on using Pydantic with
python-semver
.Holding this in a draft state until the
_VersionPydanticAnnotation
is included in the nextpydantic-extra-types
release, probably version2.10.0
.Any and all feedback appreciated, cheers.