Skip to content

fix: Update Python site-packages path to version 3.14 #64

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.

Sign up for GitHub

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

Merged
venkatamutyala merged 1 commit into from
Mar 3, 2026
Merged

fix: Update Python site-packages path to version 3.14 #64

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 2 additions & 2 deletions Dockerfile
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -17,7 +17,7 @@ FROM python:3.14.3-alpine@sha256:faee120f7885a06fcc9677922331391fa690d911c020abb
WORKDIR /app

# Copy the dependencies from the builder stage
COPY --from=builder /usr/local/lib/python3.11/site-packages /usr/local/lib/python3.11/site-packages
COPY --from=builder /usr/local/lib/python3.14/site-packages /usr/local/lib/python3.14/site-packages
Copy link

Copilot AI Mar 3, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The Python minor version is hardcoded in the site-packages copy path (python3.14). Since the base image tag is also hardcoded (3.14.3) this creates duplication and makes it easy to reintroduce a mismatch (as happened previously with python3.11). Consider introducing a single build arg for the Python minor version and using it consistently for both the image tag and the COPY paths, or using a version-agnostic install/copy pattern (e.g., pip install to a dedicated prefix directory and copy that).

Copilot uses AI. Check for mistakes.
COPY --from=builder /usr/local/bin /usr/local/bin

# Copy the application files
Expand All @@ -43,4 +43,4 @@ USER appuser
EXPOSE 8000

# Use ENTRYPOINT to ensure the container runs as expected
ENTRYPOINT ["python", "-u", "main.py"]
ENTRYPOINT ["python", "-u", "main.py"]