Fix a typo in the DeprecationWarning warning of _VersionPydanticAnnotation #289
Conversation
Signed-off-by: urasakikeisuke <[email protected]>
| @@ -13,7 +13,8 @@ | |||
| from typing_extensions import Annotated | |||
|
|
|||
| warnings.warn( | |||
| 'Use from pydantic_extra_types.semver import SemanticVersion instead. Will be removed in 3.0.0.', DeprecationWarning | |||
| 'Use from pydantic_extra_types.semantic_version import SemanticVersion instead. Will be removed in 3.0.0.', | |||
There was a problem hiding this comment.
Hello @urasakikeisuke the file name is semver so changing this pydantic_extra_types.semantic_version require changing the name of the file
There was a problem hiding this comment.
Hello @yezz123! The guidance in the warning “from pydantic_extra_types.semver import SemanticVersion” led to the following ImportError.
>>> from pydantic_extra_types.semver import SemanticVersion
Traceback (most recent call last):
File "<python-input-0>", line 1, in <module>
from pydantic_extra_types.semver import SemanticVersion
ImportError: cannot import name 'SemanticVersion' from 'pydantic_extra_types.semver' (/home/user/.venv/lib/python3.13/site-packages/pydantic_extra_types/semver.py)
When I imported as follows, the error did not occur.
>>> from pydantic_extra_types.semantic_version import SemanticVersion
>>>
The SemanticVersion class is in semantic_version.py so the change is fine I think.
There was a problem hiding this comment.
I guess you miss changing the files here is the file you should update https://github.com/urasakikeisuke/pydantic-extra-types/blob/main/pydantic_extra_types/semantic_version.py
There was a problem hiding this comment.
Here is my take here, we can merge both of them because they are duplicated logic and we can keep one of them
This PR fixes a typo in the deprecation warning message of _VersionPydanticAnnotation.