-
-
Notifications
You must be signed in to change notification settings - Fork 5.2k
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
🐛 Fix Poetry installation in Dockerfile and upgrade Python version and packages to fix Docker build #480
Conversation
@little7Li, thanks for the PR, I wanted to do it myself. I'd change the title to something like "Upgrade poetry installer to v1.2" to reflect more accurately the changes. The previous installer for poetry 1.1 works fine, no need to fix it, just make an upgrade. |
Would be great if this were merged soon. |
Bump! The provided URL now breaks build with a 404. |
Awesome, thank you @little7Li! 😄 🍰 I updated the Python version as well as it was not building the Docker image, and I updated a couple of other dependencies needed for it to build and run. 🤓 Thanks @hcristea for the comment! ☕ |
…d packages to fix Docker build (fastapi#480) Co-authored-by: lidong293 <lidong310@jd.com> Co-authored-by: Sebastián Ramírez <tiangolo@gmail.com>
The poetry installation url has to be updated as the old one is no longer valid.
And replace a poetry install option
--no-dev
with a recommended one--only main
.