-
Notifications
You must be signed in to change notification settings - Fork 6
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
Pin known compatible versions of dependencies #275
Conversation
DiamondJoseph
commented
Jul 7, 2023
•
edited
Loading
edited
- Pins Pydantic to 1.10 and Closes Pin pydantic to v1 #270
- Dodal and our pinned version of typing_extensions both conflict with Pydantic 2.0
- Pins FastAPI to 0.99: supports OpenAPI 3.1 while using Pydantic 1.0
- Pin Bluesky to 1.10 prior to AreaDetector staging change
- Pin ScanSpec to version compatible with Pydantic and FastAPI
- Pin Click to version compatible with mypy type checking
ea763d9
to
b22af94
Compare
- Closes #270 - Dodal and our pinned version of typing_extensions both conflict with Pydantic 2.0 - Pins FastAPI to 0.99: supports OpenAPI 3.1 while using Pydantic 1.0 - Pin Bluesky to 1.10 prior to AreaDetector staging change - Pin ScanSpec to version compatible with Pydantic and FastAPI - Pin Click to version compatible with mypy type checking
f6a4c8c
to
bf1dd37
Compare
Codecov Report
@@ Coverage Diff @@
## main #275 +/- ##
=======================================
Coverage 89.44% 89.44%
=======================================
Files 41 41
Lines 1412 1412
=======================================
Hits 1263 1263
Misses 149 149 📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more |
"ophyd", | ||
"nslsii", | ||
"pyepics", | ||
"pydantic", | ||
"pydantic<2.0", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should we not allow installing v2 and change all imports to v1?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Let me see how it behaves with various versions of FastAPI