Skip to content

Adding the Dockerfile for better deployment and udating instructions#88

Merged
lotif merged 6 commits intomainfrom
marcelo/dockerfile
Apr 1, 2026
Merged

Adding the Dockerfile for better deployment and udating instructions#88
lotif merged 6 commits intomainfrom
marcelo/dockerfile

Conversation

@lotif
Copy link
Copy Markdown
Collaborator

@lotif lotif commented Apr 1, 2026

PR Type

Feature

Short Description

  • Adding Dockerfile for easier app startup and deployment
  • Upgrading outdated openai package version
  • Updating NGINX config to make docs work properly in the context of docker
  • Updating README instructions

Tests Added

NA

Summary by CodeRabbit

Release Notes

  • Documentation

    • Added comprehensive guide for local development including environment setup and dependency configuration
    • Updated primary deployment documentation to emphasize Docker-based approach with new service endpoints
  • Chores

    • Introduced Docker containerization support with Nginx reverse proxy and automated orchestration of UI and API services

@coderabbitai
Copy link
Copy Markdown

coderabbitai bot commented Apr 1, 2026

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Free

Run ID: 04a11db5-0134-4dac-8a97-a7e22165956d

📥 Commits

Reviewing files that changed from the base of the PR and between f456763 and 6785083.

⛔ Files ignored due to path filters (1)
  • uv.lock is excluded by !**/*.lock
📒 Files selected for processing (5)
  • CONTRIBUTING.md
  • Dockerfile
  • README.md
  • docker_services.sh
  • src/aspis/nginx/nginx.conf

📝 Walkthrough

Walkthrough

This pull request introduces Docker containerization for the application alongside updated documentation. A new Dockerfile builds an Ubuntu-based container with Python, Nginx, and application dependencies using uv. A docker_services.sh entrypoint script launches both the FastAPI server (port 8000) and Streamlit UI (port 8501) in the background, then runs Nginx in the foreground. The Nginx configuration was extended with new proxy routes for /docs and /openapi.json endpoints. Documentation was updated to reflect Docker-based setup in the README, while a new "Running From Source" section was added to CONTRIBUTING.md with local development instructions.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~22 minutes


Note

🎁 Summarized by CodeRabbit Free

Your organization is on the Free plan. CodeRabbit will generate a high-level summary and a walkthrough for each pull request. For a comprehensive line-by-line review, please upgrade your subscription to CodeRabbit Pro by visiting https://app.coderabbit.ai/login.

Comment @coderabbitai help to get the list of available commands and usage tips.

@lotif lotif merged commit 4a360ed into main Apr 1, 2026
9 checks passed
@lotif lotif deleted the marcelo/dockerfile branch April 1, 2026 22:34
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.

1 participant