Skip to content

Make it happen#12

Open
iMartzen wants to merge 13 commits intomainfrom
make-it-happen
Open

Make it happen#12
iMartzen wants to merge 13 commits intomainfrom
make-it-happen

Conversation

@iMartzen
Copy link
Owner

@iMartzen iMartzen commented Feb 3, 2025

This pull request includes significant updates to the Dockerfile, Makefile, and README.md files, as well as the removal of the run.sh and stop.sh scripts. The changes aim to streamline the setup process, improve documentation, and enhance the Docker configuration.

Docker and Makefile updates:

  • Dockerfile: Updated to use docker.io/library/python:3.9-slim and reorganized the commands for better readability.
  • Makefile: Introduced new variables and commands to manage Docker Compose operations, replacing the need for run.sh and stop.sh scripts.
  • docker-compose.yml: Modified to use environment variables for ports and hostname, improving flexibility and configurability.

Documentation improvements:

  • README.md: Enhanced instructions for local setup, including steps to generate a .env file and use make commands instead of shell scripts. Added clarity on environment variables and updated API endpoint descriptions. [1] [2] [3] [4] [5] [6] [7] [8] [9] [10]

Removal of obsolete scripts:

  • run.sh and stop.sh: These scripts were removed in favor of the new Makefile commands, simplifying the process of running and stopping the application. [1] [2]

@vercel
Copy link

vercel bot commented Feb 3, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
vogel-spotter ✅ Ready (Inspect) Visit Preview 💬 Add feedback Feb 3, 2025 8:03pm

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.

2 participants