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

Upgrade dependencies, Pydantic v2 #796

Merged
merged 15 commits into from
Sep 4, 2023
Merged

Conversation

spwoodcock
Copy link
Member

  • Pydantic v2 is a major breaking change, required for later versions of FastAPI.
  • I updated syntax to be in line with v2.
  • Also updated all other dependencies to latest.

Note

@spwoodcock spwoodcock added priority:high Should be addressed as a priority backend Related to backend code labels Sep 1, 2023
@spwoodcock spwoodcock self-assigned this Sep 1, 2023
@spwoodcock spwoodcock changed the base branch from main to development September 1, 2023 15:21
@robsavoye
Copy link
Collaborator

Can you fix the conflicts ?

@spwoodcock spwoodcock force-pushed the build/upgrade-pydantic-v2 branch from 4e5dbd2 to f4d8653 Compare September 1, 2023 16:41
@spwoodcock spwoodcock temporarily deployed to 796/merge September 1, 2023 16:41 — with GitHub Actions Inactive
@spwoodcock spwoodcock temporarily deployed to 796/merge September 1, 2023 16:55 — with GitHub Actions Inactive
@spwoodcock spwoodcock marked this pull request as ready for review September 1, 2023 16:56
@spwoodcock
Copy link
Member Author

Documented failing pytest workflow here: #797
Will fix as soon as I get a chance.

This is ready to merge though.

@spwoodcock spwoodcock force-pushed the build/upgrade-pydantic-v2 branch from b188b74 to ea04fa0 Compare September 4, 2023 17:20
@robsavoye robsavoye merged commit eb025c5 into development Sep 4, 2023
1 check passed
@spwoodcock spwoodcock deleted the build/upgrade-pydantic-v2 branch September 6, 2023 07:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backend Related to backend code priority:high Should be addressed as a priority
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants