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

Update pydantic to v2 #294

Merged
merged 3 commits into from
Feb 9, 2024
Merged

Conversation

evalott100
Copy link
Contributor

Description

Changed event model to use pydantic v2. This is now pinned.

Motivation and Context

Since the release of pydantic v2, we've been doing generation code with pydantic 1. Pydantic 2 allows us to do this with less code.

How Has This Been Tested?

Tested against bluesky itself. So long as the schema match up the runtime code should work the same.

@evalott100 evalott100 self-assigned this Feb 1, 2024
@evalott100 evalott100 marked this pull request as draft February 1, 2024 14:03
@evalott100 evalott100 force-pushed the update_pydantic_to_v2 branch 8 times, most recently from c3c4d07 to 862fafc Compare February 5, 2024 09:53
@evalott100 evalott100 marked this pull request as ready for review February 5, 2024 09:58
@evalott100 evalott100 force-pushed the update_pydantic_to_v2 branch 4 times, most recently from e470fda to 70bf3d5 Compare February 5, 2024 10:21
@danielballan danielballan self-requested a review February 6, 2024 16:00
Bumps [actions/download-artifact](https://github.com/actions/download-artifact) from 3 to 4.
- [Release notes](https://github.com/actions/download-artifact/releases)
- [Commits](actions/download-artifact@v3...v4)

---
updated-dependencies:
- dependency-name: actions/download-artifact
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <support@github.com>
@evalott100 evalott100 force-pushed the update_pydantic_to_v2 branch 4 times, most recently from 45256ac to bcc4434 Compare February 7, 2024 09:45
event_model/documents/__init__.py Show resolved Hide resolved
event_model/documents/generate/type_wrapper.py Outdated Show resolved Hide resolved
pyproject.toml Outdated Show resolved Hide resolved
The schema generation code now uses pydantic 2,
which has been pinned in the `pyproject.toml`.
@danielballan danielballan merged commit 65d14ca into bluesky:main Feb 9, 2024
9 checks passed
@evalott100 evalott100 deleted the update_pydantic_to_v2 branch February 9, 2024 13:26
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