Skip to content

Update requirements #48

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
GitTimeraider merged 8 commits into from
Oct 13, 2025
Merged

Update requirements #48

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
8 commits
Select commit Hold shift + click to select a range
13887af
Bump python from 3.13-slim to 3.14-slim
dependabot[bot] Oct 13, 2025
3a6b4be
Bump pymysql from 1.1.1 to 1.1.2
dependabot[bot] Oct 13, 2025
1ad8975
Bump psycopg[binary,pool] from 3.2.3 to 3.2.10
dependabot[bot] Oct 13, 2025
c7e360c
Merge pull request #47 from GitTimeraider/dependabot/pip/psycopg-bina…
GitTimeraider Oct 13, 2025
bf7d64d
Merge branch 'develop' into dependabot/pip/pymysql-1.1.2
GitTimeraider Oct 13, 2025
bf57d8f
Merge pull request #46 from GitTimeraider/dependabot/pip/pymysql-1.1.2
GitTimeraider Oct 13, 2025
3b72f60
Merge pull request #45 from GitTimeraider/dependabot/docker/python-3.…
GitTimeraider Oct 13, 2025
4c688d4
Update python
GitTimeraider Oct 13, 2025
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
8 changes: 4 additions & 4 deletions Dockerfile
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
# Build stage
FROM python:3.13-slim as builder
FROM python:3.14-slim as builder

Check warning on line 2 in Dockerfile

View workflow job for this annotation

GitHub Actions / build

The 'as' keyword should match the case of the 'from' keyword 

FromAsCasing: 'as' and 'FROM' keywords' casing do not match
More info: https://docs.docker.com/go/dockerfile/rule/from-as-casing/

Check warning on line 2 in Dockerfile

View workflow job for this annotation

GitHub Actions / build

The 'as' keyword should match the case of the 'from' keyword 

FromAsCasing: 'as' and 'FROM' keywords' casing do not match
More info: https://docs.docker.com/go/dockerfile/rule/from-as-casing/

WORKDIR /app

Expand All @@ -19,7 +19,7 @@
RUN pip install --no-cache-dir -r requirements.txt

# Runtime stage
FROM python:3.13-slim
FROM python:3.14-slim

WORKDIR /app

Expand All @@ -39,8 +39,8 @@
&& mkdir -p /app/instance \
&& chown -R appuser:appgroup /app

# Copy installed packages from builder stage
COPY --from=builder /usr/local/lib/python3.13/site-packages /usr/local/lib/python3.13/site-packages
# Copy installed packages from builder stage (version-agnostic)
COPY --from=builder /usr/local/lib/ /usr/local/lib/
Copy link

Copilot AI Oct 13, 2025

Choose a reason for hiding this comment

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

Copying the entire /usr/local/lib/ directory may include unnecessary files and increase image size. The previous version-specific approach was more precise. Consider using a more targeted copy pattern like /usr/local/lib/python*/site-packages to maintain version-agnostic behavior while avoiding unnecessary files.

Suggested change
COPY --from=builder /usr/local/lib/ /usr/local/lib/
COPY --from=builder /usr/local/lib/python*/site-packages /usr/local/lib/python*/site-packages

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

# Copy application files and set ownership
Expand Down
4 changes: 2 additions & 2 deletions requirements.txt
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -11,5 +11,5 @@ Werkzeug==3.1.3
requests==2.32.5

# Database drivers
psycopg[binary,pool]==3.2.3
PyMySQL==1.1.1
psycopg[binary,pool]==3.2.10
PyMySQL==1.1.2