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

added bytes type for Pydantic #31

Merged
merged 5 commits into from
Apr 21, 2022
Merged

added bytes type for Pydantic #31

merged 5 commits into from
Apr 21, 2022

Conversation

vortex14
Copy link
Contributor

Added support for the byte type for the Pydantic model


class User(BaseModel):
    avatar: bytes

@xnuinside
Copy link
Owner

@vortex14 hi, thanks for opening your PR! Can you add small test to be sure, that this mapping do exactly that expected?

@vortex14
Copy link
Contributor Author

@vortex14 hi, thanks for opening your PR! Can you add small test to be sure, that this mapping do exactly that expected?

it's ready

@xnuinside
Copy link
Owner

@vortex14 pipeline is failed :) small flake8 issue - https://github.com/xnuinside/omymodels/runs/6111686198?check_suite_focus=true

@xnuinside
Copy link
Owner

@vortex14 not sure what is with pipeline) I will merge PR and fix this pipeline issue later. Thank you for the PR!

@xnuinside xnuinside merged commit 7a2a960 into xnuinside:main Apr 21, 2022
@xnuinside
Copy link
Owner

@vortex14 released in 0.11.1 version

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.

2 participants