-
Notifications
You must be signed in to change notification settings - Fork 34
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
Feat/validate json update #73
Conversation
you can use https://pypi.org/project/singledispatchmethod/ for python <3.8 |
db27a69
to
58b3138
Compare
I tried on local, same error for 3.7. |
geojson_pydantic/geometries.py
Outdated
@@ -6,6 +6,7 @@ | |||
|
|||
from pydantic import BaseModel, Field, ValidationError, validator | |||
from pydantic.error_wrappers import ErrorWrapper | |||
from singledispatchmethod import singledispatchmethod |
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.
this needs to be optional
is because the tests fails directly for 3.7 so GitHub actions kills all the other matrices |
This libraries fails on 3.7. I will investigate, but later this week. |
closing this. See #66 (comment) |
What I am changing
Geometry
to increase readability of specific type validation.How I did it
functools.singledispatchmethod
.How you can test it
Important notes
This is for
Python >= 3.9
. python/cpython#83860