Skip to content

added bytes type for Pydantic - #31

Merged
xnuinside merged 5 commits into
xnuinside:mainfrom
vortex14:main
Apr 21, 2022
Merged

xnuinside merged 5 commits into
xnuinside:mainfrom
vortex14:main

Conversation

@vortex14

Copy link
Copy Markdown
Contributor

Added support for the byte type for the Pydantic model


class User(BaseModel):
    avatar: bytes

@xnuinside

Copy link
Copy Markdown
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
Copy Markdown
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
Copy Markdown
Owner

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

@xnuinside

Copy link
Copy Markdown
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
Copy Markdown
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