fix: Constrain the minimum bound of pydantic. #389
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Alternatively, I could alter your uses of
Field()
which do not set adefault
value. But the code as-is appears to be incompatible with pydantic before 1.5, where it raises errors like:Likely for all these cases, the default could be set to
...
, but you have a decent number ofField
usages, and this seemed less invasive a solution reflecting the current state of the code base.