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.

Sign up for GitHub

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

Jump to bottom

feat: upgrade pydantic to v2 #20

Merged
merged 1 commit into from
Jul 12, 2024
Merged

feat: upgrade pydantic to v2 #20

merged 1 commit into from
Jul 12, 2024

Conversation

zhongzi
Copy link
Collaborator

@zhongzi zhongzi commented Jul 5, 2024

No description provided.

@zhongzi zhongzi force-pushed the v2 branch 5 times, most recently from 8cc9430 to 877b6b6 Compare July 5, 2024 10:34
@zhongzi
Copy link
Collaborator Author

zhongzi commented Jul 9, 2024 •
edited
Loading

没什么太大问题就合并？🙏

@zhongzi
Copy link
Collaborator Author

zhongzi commented Jul 12, 2024 •
edited
Loading

这一版本应该都到位：

  • schame定义与重定义调整，解决Relation和require状态在ORM到pydantic之间转化关系调整
    • async计算字段将自动调整为非计算字段，带默认值，可选类型
  • 数据映射转化过滤调整，比如不可序列化字段过滤、unnullable&default中None的转化处理等
    • 这一部分两个出入两个节点都有涉及
    • ORM转pydantic入口：__init__和model_validate
    • pydantic转json出口：model_dump

@zhongzi zhongzi merged commit 4bc9fe4 into 42signal:master Jul 12, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@exherb exherb exherb left review comments

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@zhongzi @exherb