-
Notifications
You must be signed in to change notification settings - Fork 15
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 python #647
Update python #647
Conversation
Codecov Report
@@ Coverage Diff @@
## main #647 +/- ##
=======================================
Coverage 96.37% 96.37%
=======================================
Files 45 45
Lines 2538 2539 +1
=======================================
+ Hits 2446 2447 +1
Misses 92 92
Flags with carried forward coverage won't be shown. Click here to find out more. |
phdi | ||
pydantic==1.10.9 | ||
fastapi==0.96.0 | ||
phdi @ git+https://github.com/CDCgov/phdi@updatepython |
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.
change this before merging
typing_extensions==4.6.3 | ||
pydantic==1.10.9 | ||
fastapi==0.96.0 | ||
phdi @ git+https://github.com/CDCgov/phdi@updatepython |
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.
also change before merging
phdi | ||
pydantic==1.10.9 | ||
fastapi==0.96.0 | ||
phdi @ git+https://github.com/CDCgov/phdi@updatepython |
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.
another
phdi @ git+https://github.com/CDCgov/phdi@main | ||
pydantic==1.10.9 | ||
fastapi==0.96.0 | ||
phdi @ git+https://github.com/CDCgov/phdi@updatepython |
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.
and here
phdi | ||
pydantic==1.10.9 | ||
fastapi==0.96.0 | ||
phdi @ git+https://github.com/CDCgov/phdi@updatepython |
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.
and another
phdi | ||
pydantic==1.10.9 | ||
fastapi==0.96.0 | ||
phdi @ git+https://github.com/CDCgov/phdi@updatepython |
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.
here
Update Python
Summary
Updating python to 3.11 and related packages. There is a slight workaround where we have to point the repo to this branch as we cut the new release of phdi to pypi. The containers on this project are dependent on the pypi version of phdi but since those are still on the old packages, it creates a somewhat circular logic when updating python.
For now, we can point the phdi package to this github branch and then when we make a new release, we have to reupdate the release to point it back to pypi.
Related Issue
Fixes #399