Skip to content

feat: update dependencies - #42

Merged
aschuma merged 2 commits into
mainfrom
modernizer
Jul 24, 2026
Merged

feat: update dependencies#42
aschuma merged 2 commits into
mainfrom
modernizer

Conversation

@aschuma

@aschuma aschuma commented Jul 24, 2026

Copy link
Copy Markdown
Owner

No description provided.

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Updates the runtime/container build and Python dependencies to newer versions, and adds a simple HTTP health endpoint to support container health probing.

Changes:

  • Bump Python dependencies in requirements.txt (including Flask/Werkzeug and vvspy).
  • Rework the Docker image to Python 3.13 on Debian bookworm with a multi-stage venv build, non-root runtime user, and a container HEALTHCHECK.
  • Add a /health endpoint and introduce a .dockerignore to reduce Docker build context.

Reviewed changes

Copilot reviewed 3 out of 4 changed files in this pull request and generated 2 comments.

File Description
requirements.txt Updates pinned dependency versions.
Dockerfile Moves to Python 3.13 bookworm, builds deps into a venv in a builder stage, runs as non-root, and adds a healthcheck.
app.py Adds a /health route returning a JSON response for container health probing.
.dockerignore Excludes VCS metadata and common non-runtime files from Docker build context.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread Dockerfile
Comment thread Dockerfile Outdated
@aschuma
aschuma merged commit 963340f into main Jul 24, 2026
@aschuma
aschuma deleted the modernizer branch July 24, 2026 21:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants